Your fix looks fine to me. Thanks.
> On 1 Jul 2015, at 18:04, Ivan Gerasimov <ivan.gerasi...@oracle.com> wrote: > > Hello! > > Would someone please help review this small cleanup fix? > I'll go on vacation in a week, and I would like to push it before then. > > Sincerely yours, > Ivan > > On 19.06.2015 22:17, Ivan Gerasimov wrote: >> Hello everyone! >> >> The changeset consists of two parts: >> - checking systematically for NULL result of FindClass() and GetMethodID(), >> - calling CryptReleaseContext() after CryptDestroyHash(), as it is suggested >> in MSDN [1]. >> >> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8098854 >> WEBREV: http://cr.openjdk.java.net/~igerasim/8098854/00/webrev/ >> >> [1] >> https://msdn.microsoft.com/en-us/library/windows/desktop/aa380268(v=vs.85).aspx >> >> Sincerely yours, >> Ivan >> >> >> >
