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

[Seán Coffey]

Thanks for the review Mandy. This looks good for 7u-dev Daniel. Approved.

regards,
Sean.

On 09/12/2013 22:09, Mandy Chung wrote:
> 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
>>
>