RFR: 8258077: Using -Xcheck:jni can lead to a double-free after JDK-8193234

[Mauro Lacy]

On Fri, 8 Jan 2021 03:50:10 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Hi David,
>> 
>> Thanks for researching this thoroughly.
>> What you propose is reasonable, given the circumstances: undocumented or badly documented usage in the specs, and a lot of existing code (even in books, etc) that use `JNI_COMMIT` basically as `0`. We faced the same weeks ago, when looking for examples, documentation, and use cases for this functionality.
>> 
>> A couple comments:
>> - Given that `JNI_COMMIT` is then effectively useless, maybe a good idea would be to deprecate it. This will break existing code, true, but a deprecation notice can be used, by example in the form of a compilation warning or so, for code that calls / uses it. Same in the docs, i. e. this is kept for historical reasons but will be deprecated soon, and so on.
>> - I think in this case the best for us would be to simply drop support for `JNI_COMMIT` in the Rust JNI library. At least, for the ArrayCritical methods.
>
> Hi @maurolacy can you please close this PR in favour of https://github.com/openjdk/jdk/pull/1816. Thanks.

Closing in favour of #1816.

-------------

PR: https://git.openjdk.java.net/jdk/pull/1697