Native methods are defined in Java and written in traditional languages such as C and C++ \ [[JNI 2006|AA. Bibliography#JNI 06]\]. The added extensibility comes at the cost of flexibility and portability as because the code no longer conforms to the policies enforced by Java. In the past, native methods were used for performing platform specific operations, interfacing with legacy library code and improving program performance \[[Bloch 2008|AA. Bibliography#Bloch 08]\]. Although this is no longer completely true --- because of poor portability, safety and (quite ironically) performance issues --- native code is still used to interface with legacy code.Native methods have been used for performing platform-specific operations, interfacing with legacy library code, and improving program performance [Bloch 2008]. Wiki Markup
Defining a wrapper method facilitates installing appropriate security manager checks, performing input validation before passing the arguments to the native code or when obtaining validating arguments passed to native code, validating return values, defensively copying mutable inputs, and sanitizing user input. Therefore untrusted data. Consequently, every native method must be private , and must be invoked only by a wrapper method.
Noncompliant Code Example
In this noncompliant code example, the nativeOperation()
method is both native and public; hence consequently, untrusted callers may invoke it. Native method invocations bypass security manager checks.
The example does include a wrapper; This example includes the doOperation()
wrapper method, which invokes the nativeOperation()
native method but fails to provide input validation or security checks.
Code Block | ||
---|---|---|
| ||
public final class NativeMethod { // publicPublic native method public native void nativeOperation(byte[] data, int offset, int len); // wrapperWrapper method that lacks security checks and input validation public void doOperation(byte[] data, int offset, int len) { nativeOperation(data, offset, len); } static { // loadLoad native library in static initializer of class System.loadLibrary("NativeMethodLib"); } } |
Compliant Solution
This compliant solution declares the native method private. Furthermore, the The doOperation()
wrapper method performs routine permission checking to determine whether the succeeding operations are permitted to continue. This is followed by the creation of checks permissions, creates a defensive copy of the mutable input array data
as well as by range checking , and checks the ranges of the parametersarguments. The nativeOperation()
method is thus consequently called with safe secure inputs. Note that the validation checks must produce outputs that conform to the input requirements of the native implementations/librariesmethods.
Code Block | ||
---|---|---|
| ||
public final class NativeMethodWrapper { // privatePrivate native method private native void nativeOperation(byte[] data, int offset, int len); // wrapperWrapper method performs SecurityManager and input validation checks public void doOperation(byte[] data, int offset, int len) { // permissionPermission needed to invoke native method securityManagerCheck(); if (data == null) { throw new NullPointerException(); } // copyCopy mutable input data = data.clone(); // validateValidate input if ((offset < 0) || (len < 0) || (offset > (data.length - len))) { throw new IllegalArgumentException(); } nativeOperation(data, offset, len); } static { // loadLoad native library in static initializer of class System.loadLibrary("NativeMethodLib"); } } |
Exceptions
JN100-J-EX0: Native methods that do not require security manager checks, validation of arguments or return values, or defensive copying of mutable inputs (for example, the standard C function int rand(void)
) do not need to be wrapped.
Risk Assessment
Failure to define wrappers around native methods can allow unprivileged callers to invoke them and thus exploit inherent vulnerabilities such as those resulting from invalid inputsbuffer overflows in native libraries.
Rule | Severity | Likelihood | Remediation Cost | Priority | Level |
---|
JNI00-J |
Medium |
Probable |
High | P4 | L3 |
Automated Detection
...
Automated detection is not feasible in the fully general case. However, an approach similar to Design Fragments \[ [Fairbanks 07|AA. Bibliography#Fairbanks 07]\] could assist both programmers and static analysis tools.
Related Vulnerabilities
Search for vulnerabilities resulting from the violation of this rule on the CERT website.
Related Guidelines
<ac:structured-macro ac:name="unmigrated-wiki-markup" ac:schema-version="1" ac:macro-id="9d84b64b-a052-44a2-bb2d-b1d763a66fff"><ac:plain-text-body><![CDATA[ | [[MITRE 2009 | AA. Bibliography#MITRE 09]] | [CWE ID 111 | http://cwe.mitre.org/data/definitions/111.html] "Direct Use of Unsafe JNI" | ]]></ac:plain-text-body></ac:structured-macro> |
Bibliography
2007] could assist both programmers and static analysis tools.
Tool | Version | Checker | Description | ||||||
---|---|---|---|---|---|---|---|---|---|
Parasoft Jtest |
| CERT.JNI00.NATIW | Use wrapper methods to secure native methods |
Related Guidelines
CWE-111, Direct Use of Unsafe JNI | |
Guideline 5-3 / INPUT-3: Define wrappers around native methods |
Bibliography
[JNI 2006] | |
<ac:structured-macro ac:name="unmigrated-wiki-markup" ac:schema-version="1" ac:macro-id="a7ff67de-c455-4f9f-a829-25e17796fe66"><ac:plain-text-body><![CDATA[
[[Fairbanks 2007
AA. Bibliography#Fairbanks 07]]
]]></ac:plain-text-body></ac:structured-macro>
<ac:structured-macro ac:name="unmigrated-wiki-markup" ac:schema-version="1" ac:macro-id="59016abb-e109-4d50-b6de-737b7e7ed9cf"><ac:plain-text-body><![CDATA[
[[JNI 2006
AA. Bibliography#JNI 06]]
]]></ac:plain-text-body></ac:structured-macro>
<ac:structured-macro ac:name="unmigrated-wiki-markup" ac:schema-version="1" ac:macro-id="27c7d71b-15f6-46d3-a928-bc762fb087c0"><ac:plain-text-body><![CDATA[
[[Liang 1997
AA. Bibliography#Liang 97]]
]]></ac:plain-text-body></ac:structured-macro>
<ac:structured-macro ac:name="unmigrated-wiki-markup" ac:schema-version="1" ac:macro-id="edecb0b7-7aca-4ffd-bf95-1e37cba78f5f"><ac:plain-text-body><![CDATA[
[[Macgregor 1998
] | Section 2.2.3, "Interfaces and Architectures |
]]></ac:plain-text-body></ac:structured-macro>
" |
[[SCG 2007
AA. Bibliography#SCG 07]]
Guideline 3-3 Define wrappers around native methods
]]></ac:plain-text-body></ac:structured-macro>
...
SEC13-J. Do not allow unauthorized construction of classes in inaccessible packages 14. Platform Security (SEC) SEC19-J. Do not rely on the default automatic signature verification provided by URLClassLoader and java.util.jar