Re: Review request for 8146317: Memory leak in wcstombsdmp

[Semyon Sadetsky]

Looks good.

--Semyon

On 1/6/2016 4:06 PM, Ambarish Rapte wrote:

Hi ,

                Please review the fix for JDK9,

                Bug: https://bugs.openjdk.java.net/browse/JDK-8146317

Webrev: http://cr.openjdk.java.net/~arapte/8146317/webrev.00/ <http://cr.openjdk.java.net/%7Earapte/8146317/webrev.00/>

Issue: Possible memory leak with an error case in function wcstombsdmp().

Cause:  In error case,

Function wcstombsdmp() returns NULL, without freeing memory allocated using malloc.

Fix:         Call   free() before returning NULL.

Thanks,

Ambarish