<AWT Dev> 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
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20161221/277e77a8/attachment.html>