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

Wed, 10 Jul 2013 00:44:25 -0700 

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
























					Reply via email to