Re: RFR JDK-8171836: Memory leak in java.desktop/unix/native/common/awt/fontpath.c

[Vadim Pakhnushev]

Abhijit,
I think there's some misunderstanding here.
The pointer you are trying to free is NULL already:

     if ( *newFontPath *== NULL ) {
       free ( ( void *) appendDirList );
+      free((void*) *newFontPath*);

Thanks,
Vadim

On 21.12.2016 16:02, Abhijit Roy wrote:

Hi all,

Please review the fix for the bug below:

Bug:https://bugs.openjdk.java.net/browse/JDK-8171836
Description: Memory leak in java.desktop/unix/native/common/awt/fontpath.c

Webrev:http://cr.openjdk.java.net/~rpatil/8171836/webrev.00/ <http://cr.openjdk.java.net/%7Erpatil/8171836/webrev.00/>

To prevent memory leak issue, I have released the newFontPath in 
java.desktop/unix/native/common/awt/fontpath.
Moving forward it for review.

Regards,

Abhijit