Hi All,

There is a potential problem in jdk/src/share/native/com/sun/java/util/jar/pack/jni.cpp.

(Maybe it is not suitable for posting this on core-lib, anyone could tell me which mailing list is prefer?)

L85:

JNI_GetCreatedJavaVMs(&vm, 1, null) in which the 3rd parameter is a pointer to an integer. See[1], the latest JNI Invocation API spec does not say anything about allowing a null as the last parameter.

I think it is more reasonable to change null to an integer variable. Here is my fix[2]

[1] http://docs.oracle.com/javase/7/docs/technotes/guides/jni/spec/invocation.html#wp633

[2] http://cr.openjdk.java.net/~littlee/ojdk-432/webrev.00/ <http://cr.openjdk.java.net/%7Elittlee/ojdk-432/webrev.00/>


Please review this mail!

Thanks a lot!

--
Best Regards,

Deven

Reply via email to