[10] RFR 8068024 : Null pointer dereference in jdk/src/macosx/native/apple/security/KeystoreImpl.m

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[10] RFR 8068024 : Null pointer dereference in jdk/src/macosx/native/apple/security/KeystoreImpl.m

Ivan Gerasimov
Hello!

Here's a simple fix to correctly handle the malloc call returning NULL.

BUGURL: https://bugs.openjdk.java.net/browse/JDK-8068024
WEBREV: http://cr.openjdk.java.net/~igerasim/8068024/00/webrev/

Would you please help review?

--
With kind regards,
Ivan Gerasimov

Reply | Threaded
Open this post in threaded view
|

Re: [10] RFR 8068024 : Null pointer dereference in jdk/src/macosx/native/apple/security/KeystoreImpl.m

Weijun Wang
This looks fine to me.

Thanks
Max

> On Nov 22, 2017, at 9:47 AM, Ivan Gerasimov <[hidden email]> wrote:
>
> Hello!
>
> Here's a simple fix to correctly handle the malloc call returning NULL.
>
> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8068024
> WEBREV: http://cr.openjdk.java.net/~igerasim/8068024/00/webrev/
>
> Would you please help review?
>
> --
> With kind regards,
> Ivan Gerasimov
>

Reply | Threaded
Open this post in threaded view
|

Re: [10] RFR 8068024 : Null pointer dereference in jdk/src/macosx/native/apple/security/KeystoreImpl.m

Ivan Gerasimov
Thank you for review!  Pushed.

With kind regards,
Ivan

On 11/21/17 10:32 PM, Weijun Wang wrote:

> This looks fine to me.
>
> Thanks
> Max
>
>> On Nov 22, 2017, at 9:47 AM, Ivan Gerasimov <[hidden email]> wrote:
>>
>> Hello!
>>
>> Here's a simple fix to correctly handle the malloc call returning NULL.
>>
>> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8068024
>> WEBREV: http://cr.openjdk.java.net/~igerasim/8068024/00/webrev/
>>
>> Would you please help review?
>>
>> --
>> With kind regards,
>> Ivan Gerasimov
>>
>

--
With kind regards,
Ivan Gerasimov