RFR(S): 8216426: Usage of array placement new may lead to memory corruption
Robbin Ehn
robbin.ehn at oracle.com
Tue Jan 15 08:03:04 UTC 2019
Hi Martin,
The Bucket should extend CHeapObj<F>, now that it could.
Otherwise looks good, thanks.
Originally Bucket did this and I used memset, but gdb was very upset by this.
So I changed to array placement new, which meant I had to remove CHeapObj<F> if I remember correctly :)
/Robbin
On 2019-01-14 17:39, Doerr, Martin wrote:
> Hi,
>
> thanks for looking at this issue and especially for adding comments.
> Sounds like this issue should better get fixed although there are no known problems.
>
> Should we fix it by replacing the array placement new by normal placement new for each element?
> http://cr.openjdk.java.net/~mdoerr/8216426_array_placement_new/webrev.00/
>
> Best regards,
> Martin
>
More information about the hotspot-runtime-dev
mailing list