Request for review: 8019267: NPE in AbstractSaslImpl when trace level >= FINER in KRB5

[Weijun Wang]

Hi Sean

Can you take a look at the webrev for jdk7u-dev. The only difference is 
that in 7u there is no check for output == null at all so an NPE is thrown.

    http://cr.openjdk.java.net/~weijun/8019267/7u-dev/webrev.00/

Thanks
Max

On 7/9/13 8:16 PM, Sean Mullan wrote:
> Looks fine.
>
> --Sean
>
> On 07/09/2013 03:32 AM, Weijun Wang wrote:
>> Hi All
>>
>> An NPE was suppressed in jdk8 with a simple if. This new fix tries to
>> output useful info even for a null argument.
>>
>> Please review the code changes at
>>
>>     http://cr.openjdk.java.net/~weijun/8019267/webrev.01/
>>
>> The added test makes sure logging does not crash and the output is
>> informational.
>>
>> Thanks
>> Max
>