[7u60] Request for review & approval for 8025512: NPE with logging while launching webstart on jre7u40 if logging is disabled

[Mandy Chung]

Daniel,

The fix looks good and approved.

I think this test may be good to be included in JDK 9 (backport to 8u).  
But I'd like to get rid of its dependency on awt.  I wonder if this can 
be modified to create its own implementation of JavaAWTAccess in a 
similar way of the new tests you added.    This is something to consider 
if you forward port this test.  As for 7u60, what you have is good 
enough and no change is needed.

Mandy

On 12/9/2013 5:46 AM, Daniel Fuchs wrote:
> Hi,
>
> This is a request for review & approval for
> 8025512: NPE with logging while launching webstart on jre7u40
>          if logging is disabled
>
> webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8025512/webrev.00/
> bugid: https://bugs.openjdk.java.net/browse/JDK-8025512
>
> In some circumstances, an NPE can occur in logging initialization.
>
> This is because there is one place (in Logger.setParent) where access to
> 'manager' is not guarded by a if (manager == null) - as is done
> every where else.
>
> The fix is to copy over the code we already have for this method in
> JDK 8, and do:
>
> if (manager == null) {
>    manager = LogManager.getLogManager();
> }
>
> The risk should be very limited. Not guarding for manager == null
> in this place was an oversight.
>
> The test case will fail without the fix and pass with it.
>
>
> best regards,
>
> -- daniel
>