hg: jdk7/tl/jdk: 6881442: (reflect) Race condition in Class.getName()

2009-09-12 Thread martinrb
Changeset: 020a0fed38c9 Author:martin Date: 2009-09-12 15:30 -0700 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/020a0fed38c9 6881442: (reflect) Race condition in Class.getName() Summary: only read "name" field racily once Reviewed-by: darcy ! src/share/classes

Re: Race condition in Class.getName()

2009-09-11 Thread Joe Darcy
ello. The change looks fine to me. Do you want a bug filed, etc.? Yup. I will commit as soon as I get a bug + synopsis. 6881442 (reflect) Race condition in Class.getName() -Joe

Re: Race condition in Class.getName()

2009-09-11 Thread Martin Buchholz
On Fri, Sep 11, 2009 at 16:48, Joe Darcy wrote: > Martin Buchholz wrote: >> >> It is believed that the java memory model allows Class.getName() >> to return null. >> This is one of those methods with an intentional data race. >> Probably this has not been seen in practice because only >> a perverse

Re: Race condition in Class.getName()

2009-09-11 Thread Joe Darcy
Martin Buchholz wrote: It is believed that the java memory model allows Class.getName() to return null. This is one of those methods with an intentional data race. Probably this has not been seen in practice because only a perverse or adversarial runtime would load the "name" field twice, out-of-

Race condition in Class.getName()

2009-09-09 Thread Martin Buchholz
It is believed that the java memory model allows Class.getName() to return null. This is one of those methods with an intentional data race. Probably this has not been seen in practice because only a perverse or adversarial runtime would load the "name" field twice, out-of-order, as it seems to be