Looks fine to me. Don't you want to add a exception message to indicate the input stream is null?
Xuelei On 8/18/2015 7:23 PM, Vincent Ryan wrote: > Please review this trivial change to the KeyStoreSpi.engineProbe method > so that it matches its specification wrt NPE. > Thanks. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8130799 > Webrev: http://cr.openjdk.java.net/~vinnie/8130799/webrev.00/