[PATCH FOR REVIEW] Potential Buffer Overflow in java_props_md.c

Alan Bateman Alan.Bateman at oracle.com
Wed Aug 8 09:51:08 UTC 2012


On 08/08/2012 01:33, David Holmes wrote:
> On 8/08/2012 2:40 AM, Xueming Shen wrote:
>>
>> Andrew,
>>
>> Since we are here:-) are we also supposed to "free" the old_temp at #250
>> and old_temp and old_ev at the end?
>
> No. They are aliases for temp and encoding_variant, which are freed at 
> the end. There are only ever at most two things to free.
>
> Andrew: this is fine by me, but needs TL approval (Alan?)
You are a reviewer on the jdk8 project so Andrew should be all set. In 
any case, the update looks good to me too.

-Alan.



More information about the core-libs-dev mailing list