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

[Abhijit Roy]

Hi Vadim,

Yes. I did a mistake here. Please find the correct webrev below.

Webrev: http://cr.openjdk.java.net/~rpatil/8171836/webrev.01/


Thanks
Abhijit

On 12/21/2016 7:47 PM, Vadim Pakhnushev wrote:

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