8284093: Memory leak: X11SD_DisposeXImage should also free obdata

zhengyu123 · zhengyu123 · commit 205cfb84968f · 2022-04-11T13:36:20.000Z
Reviewed-by: prr, andrew
diff --git a/src/java.desktop/unix/native/common/java2d/x11/X11SurfaceData.c b/src/java.desktop/unix/native/common/java2d/x11/X11SurfaceData.c
@@ -1298,6 +1298,7 @@ void X11SD_DisposeXImage(XImage * image) {
 #ifdef MITSHM
         if (image->obdata != NULL) {
             X11SD_DropSharedSegment((XShmSegmentInfo*)image->obdata);
+            free(image->obdata);
             image->obdata = NULL;
         }
 #endif /* MITSHM */

Original file line number	Diff line number	Diff line change
`@@ -1298,6 +1298,7 @@ void X11SD_DisposeXImage(XImage * image) {`
`1298`	`1298`	`#ifdef MITSHM`
`1299`	`1299`	`if (image->obdata != NULL) {`
`1300`	`1300`	`X11SD_DropSharedSegment((XShmSegmentInfo*)image->obdata);`
	`1301`	`+ free(image->obdata);`
`1301`	`1302`	`image->obdata = NULL;`
`1302`	`1303`	`}`
`1303`	`1304`	`#endif /* MITSHM */`