Request for review- RFE 8005716 (original) (raw)

BILL PITTORE bill.pittore at oracle.com
Fri Mar 8 02:22:20 UTC 2013

Moved the string allocation into buildJniFunctionName as Alan suggested. Built and tested on windows and linux. Updated the webrev:

http://cr.openjdk.java.net/~bpittore/8005716/jdk-webrev.02/

bill

On 3/7/2013 3:50 PM, Alan Bateman wrote:

On 07/03/2013 16:36, BILL PITTORE wrote:

I updated the code based on the feedback. To fix the windows symbol name issue that Dean brought up I added a platform specific function, buildJniFunctionName. In windows it will properly convert JNIOnLoad at 8 to JNIOnLoadlibname at 8 as well as handle JNIOnLoad symbol name.

Fixed copyright headers as well. Tested on linux and windows Webrevs are here: http://cr.openjdk.java.net/~bpittore/8005716/jdk-webrev.01/ http://cr.openjdk.java.net/~bpittore/8005716/hs-webrev.00/ bill Thanks for sorting out the decorator issue. One comment on that part of the implementation is that findJniFunction is sizing the buffer and buildJniFunctionName is using strcpy in the components. I wonder if this could be changed so that buildJniFunctionName sizes the buffer and uses strncpy. That way it will be obviously safe when just looking at one function. It makes it easier for static analysis tools to check too. -Alan

More information about the core-libs-dev mailing list