commit | d4503990041e0d4164a6206d141c459caa0fde74 | [log] [tgz] |
---|---|---|
author | Nathan Hartman <hartmannathan@apache.org> | Sun Aug 16 04:39:55 2020 +0000 |
committer | Nathan Hartman <hartmannathan@apache.org> | Sun Aug 16 04:39:55 2020 +0000 |
tree | 9dff3c628b34e1afb85515c62bb3f5d2cd5e4df8 | |
parent | 8f2525c0af385032cb65e9af63045c981fa5e5e7 [diff] |
Fix crash in JavaHL JNI wrapper caused by object lifetimes See dev@ email thread "JNI segfault while running Java tests" started 2020/08/08, archived at: https://lists.apache.org/thread.html/rff3fa5ea97267adf36cc9daa3be01392e53fbc8050a91df4bbac01d8%40%3Cdev.subversion.apache.org%3E https://svn.haxx.se/dev/archive-2020-08/0010.shtml * subversion/bindings/javahl/native/jniwrapper/jni_string_map.hpp (Java::BaseImmutableMap::Entry::key): As the locally-scoped String::Contents object keeps a reference to a String, ensure the String will remain in scope for the lifetime of the String::Contents object by instantiating it as a local variable. Reported by: jamessan Patch by: Daniel Sahlberg <daniel.l.sahlberg {_AT_} gmail.com> Reviewed by: Alexandr Miloslavskiy <alexandr.miloslavskiy {_AT_} syntevo.com> hartmannathan Tested by: jamessan git-svn-id: https://svn.apache.org/repos/asf/subversion/trunk@1880886 13f79535-47bb-0310-9956-ffa450edef68