SIGSEGV in gmsec::api::jni::convertEvent about gmsec_api HOT 2 OPEN

Upon further investigation it seems that the problem is two fold.

First the code at [1] does not check the return value of FindClass before using it in another function call at [2], which causes the segfault. Specifically, FindClass will return 0 if it doesn't find the class and GetStaticFieldID will segfault if the class is null.

As a Java developer, I would expect that if a class cannot be found then a ClassNotFoundException will be thrown.

Second the code at [1] does not use the correct ClassLoader. Quoting from [3], "FindClass locates the class loader associated with the current native method; that is, the class loader of the class that declared the native method." If " there is no current native method or its associated class loader [then] in that case, the result of ClassLoader.getSystemClassLoader is used." Looking at the back trace there is no Java frames that FindClass can retrieve the ClassLoader from, so it will use the system ClassLoader, which does not contain the requested class.

As a Java developer, I would expect that if a class needs to be loaded to service a request then the ClassLoader of the class making the request will be used, or the ClassLoader of the callback.

As a Java developer, I expect the system ClassLoader will never be used.

Finally, one more request. Please include debugging symbols in your releases. Then it will be much easier to find the bugs in the native code as I won't have to disassemble it first.

[1]

[2]

[3] https://docs.oracle.com/javase/8/docs/technotes/guides/jni/spec/functions.html#FindClass

from gmsec_api.

Looking through the code a bit more, it seems that [4] has the right idea - to initialize all classes when the JNI interface is initialized. Then there will be Java frame on the stack, so FindClass will use that ClassLoader. It seems the call to FindClass should be moved form Jenv to Cache. And any other calls to load GMSEC API Java classes should be done in Cache as well so they are loaded when the JNI interface is initialized.

[4]

from gmsec_api.