Request for review- RFE 8005716

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 
>> _JNI_OnLoad at 8 to _JNI_OnLoad_libname at 8 as well as handle JNI_OnLoad 
>> 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