A nested class is any class whose declaration occurs within the body of another class or interface [JLS 20052015]. The use of a nested class is error prone unless the semantics are well understood. A common notion is that only the nested class may access the contents of the outer class. Not only does the nested class have access to the private fields of the outer class, but the same fields can be accessed by any other class within the package when the nested class is declared public or if it contains public methods or constructors. As a result, the nested class must not expose the private members of the outer class to external classes or packages.
According to The Java Language Specification (JLS), §8.3, "Field Declarations" [JLS 20052015]:
Note that a private field of a superclass might be accessible to a subclass (for example, if both classes are members of the same class). Nevertheless, a private field is never inherited by a subclass.
...
The Java language system weakens the accessibility of private members of an outer class when a nested inner class is present, which can result in an information leak.
Rule | Severity | Likelihood | Remediation Cost | Priority | Level |
---|---|---|---|---|---|
OBJ08-J | Medium | Probable | Medium | P8 | L2 |
Automated Detection
Automated detection of nonprivate inner classes that define nonprivate members and constructors that leak private data from the outer class is straightforward.
Tool | Version | Checker | Description | ||||||
---|---|---|---|---|---|---|---|---|---|
CodeSonar |
| JAVA.CLASS.ICSBS | Inner Class Should be Static (Java) | ||||||
Parasoft Jtest |
| CERT.OBJ08.INNER | Make all member classes "private" |
Related Guidelines
Bibliography
[JLS |
2015] | §8.1.3, Inner Classes and Enclosing Instances |
Section 2.3, "Inner Classes" | |
Securing Java: Getting Down to Business with Mobile Code |
...
...