Review request for 5049299
Michael McMahon
Michael.McMahon at Sun.COM
Fri May 22 16:29:38 UTC 2009
David Holmes - Sun Microsystems wrote:
> Hi Michael,
>
> > But the implementation in processhelper, does not know about JNI, so
> > it ignores the env
>
> and doesn't throw the exception, which then begs the question as to
> what happens if malloc returns NULL in that case? If I understand you
> right, the jlup_* functions called from processhelper will use its
> jlup_xmalloc function, which simply returns malloc - but those jlup_*
> functions don't check for a NULL return because they "know" that
> xmalloc throws on out-of-memory.
Right. It's only called in a couple of places, so that should be easy
enough to fix
(check for null return)
Thanks,
Michael.
More information about the core-libs-dev
mailing list