Request for review- RFE 8005716

BILL PITTORE bill.pittore at oracle.com
Mon Mar 11 15:37:01 UTC 2013


On 3/11/2013 9:40 AM, Alan Bateman wrote:
> On 08/03/2013 02:22, BILL PITTORE wrote:
>> 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
> I see this updates the method descriptions to take on board Jeremy's 
> comment on the possibility of the library being statically linked with 
> the main executable with or without the VM. To be complete, I think 
> this will require an update to the UnstatisfiedLinkError description too.
>
> Thanks for moving the sizing/allocation of the function name into 
> buildJniFunctionName as that is cleaner and safer. Is FILENAME_MAX 
> really the right limit to impose?
Maybe JVM_MAXPATHLEN is the better choice.

>
> A minor point, but the buffer is overrsized by 1 for the shared 
> library case, I think it should be:
>   len = strlen(sym)  + (cname != NULL ? strlen(cname)+1 : 0) + 1
The +2 covered both cases; but re-wrote it to be the more explicit 
version above.

bill
>
> Otherwise, I think the implementation looks good to me.
>
> -Alan.
>
>
>
>




More information about the core-libs-dev mailing list