RFR 8098854: Do cleanup in a proper order in mscapi

Ivan Gerasimov ivan.gerasimov at oracle.com
Wed Jul 1 19:01:46 UTC 2015



On 01.07.2015 20:08, Vincent Ryan wrote:
> Your fix looks fine to me.
> Thanks.
>
Thank you Vincent!


>> On 1 Jul 2015, at 18:04, Ivan Gerasimov <ivan.gerasimov at 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
>>>
>>>
>>>
>
>



More information about the security-dev mailing list