Replace the static error string in ZIP_Put_In_Cache0 with on stack memory

[Sean Chou]

Hi Alan,

    Many thanks.

    I updated the patch, ZIP_Open frees the error message and set "Zip file
open error".

The new webrev is :  http://cr.openjdk.java.net/~zhouyx/7159982/webrev.01/

    Please take a look once more.

On Thu, Apr 12, 2012 at 4:09 AM, Alan Bateman <Alan.Bateman at oracle.com>wrote:

> On 11/04/2012 15:58, Sean Chou wrote:
>
>> Hi hotspot guys,
>>
>>    Would any one like to take a look at this? I'm trying to fix a
>> potential race in ZIP_Open, it is found classLoader.cpp uses this function.
>> So a webrev for hotspot is made as well, but I need a sponsor from hotspot
>> as suggested by Alan Bateman.
>>
>>    The start of this thread is http://mail.openjdk.java.net/**
>> pipermail/core-libs-dev/2012-**April/009766.html<http://mail.openjdk.java.net/pipermail/core-libs-dev/2012-April/009766.html>.
>>    The webrevs: http://cr.openjdk.java.net/~**
>> zhouyx/7159982/webrev-hotspot.**00/<http://cr.openjdk.java.net/~zhouyx/7159982/webrev-hotspot.00/><
>> http://cr.openjdk.java.net/%**7Ezhouyx/7159982/webrev-**hotspot.00/<http://cr.openjdk.java.net/%7Ezhouyx/7159982/webrev-hotspot.00/>
>> >
>> and http://cr.openjdk.java.net/~**zhouyx/7159982/webrev.00/<http://cr.openjdk.java.net/~zhouyx/7159982/webrev.00/><
>> http://cr.openjdk.java.net/%**7Ezhouyx/7159982/webrev.00/<http://cr.openjdk.java.net/%7Ezhouyx/7159982/webrev.00/>>
>>  .
>>
>>
>>    And, I found JDK_Version::is_gte_jdk18x_**version suggested by Alan
>> does not exist in hotspot yet, shall I add it to the patch too ?
>>
> I had a brief chat with Coleen Phillimore about this. One suggestion that
> would avoid HotSpot changes is to change ZIP_Open so that it doesn't return
> an error message or returns something fixed. It's not used in the VM anyway
> (as per Coleen's reply). We need the error message for the java.util.zip
> APIs but that doesn't go through ZIP_Open so it means it can be freed and
> *pmsg = NULL returned (or a pointer to one fixed generic message).
>
> -Alan
>



-- 
Best Regards,
Sean Chou