Null pointer dereference (Android + JNI) about scapix

[Boris-Rasin]

As a workaround, you can call once in each thread created from C++:

scapix::link::java::attach_current_thread();

For convenience, I can also add generic version of this function (which will do nothing with other bridge languages):

scapix::init_thread();

I can also add optional automatic thread attachment:

target_compile_definitions(scapix PUBLIC SCAPIX_JNI_AUTO_ATTACH_THREAD)

I want to make it optional because it would add slight runtime overhead on each JNI call.

What do you think?

[nadrolinux]

Hi,
I think that 3rd option with optional auto attachments is the best, because of automatic nature, but I agree that marked it as optional is better choice, because in this way we can still rely on options 1 and/or 2 if auto performance will be a problem. Please just remember that for option no. 2 you should also add 'scapix::exit_thread' or similar with internal 'scapix::link::java::detach_current_thread()' call.
Thanks for a quick response and good solutions as always.
Patryk

[Boris-Rasin]

Commit: 0256e3f
Documentation: https://www.scapix.com/java_link/threads/

[Boris-Rasin]

Released in cmodule v1.0.35:
https://github.com/scapix-com/cmodule/releases

[nadrolinux]

Great, thanks a lot.