RESOLVED FIXED 153413
Colormap should be freed when XWindow is destroyed on X11Helper. 
https://bugs.webkit.org/show_bug.cgi?id=153413
Summary Colormap should be freed when XWindow is destroyed on X11Helper.
Sun-woo Nam
Reported 2016-01-24 18:13:20 PST
Colormap is needed to create XWindow and it should be freed when XWindow is destroyed. Unless Colormap is freed before destroying XWindow, memory leak is suspected. XFreeColormap therefore is needed on X11Helper.
Attachments
Patch (5.01 KB, patch)
2016-01-24 18:20 PST, Sun-woo Nam 		no flags
DetailsFormatted DiffDiff
Patch (1.59 KB, patch)
2016-01-24 18:22 PST, Sun-woo Nam 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Sun-woo Nam
Comment 1 2016-01-24 18:20:15 PST
Created attachment 269712 [details] Patch
Sun-woo Nam
Comment 2 2016-01-24 18:22:11 PST
Created attachment 269713 [details] Patch
Zan Dobersek
Comment 3 2016-02-01 06:12:42 PST
Comment on attachment 269713 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=269713&action=review > Source/WebCore/platform/graphics/surfaces/glx/X11Helper.cpp:296 > + XWindowAttributes attribute; > + XGetWindowAttributes(display, windowId, &attribute); > + > + XFreeColormap(display, attribute.colormap); r=me
Michael Catanzaro
Comment 4 2016-02-01 15:58:59 PST
Comment on attachment 269713 [details] Patch Thanks!
WebKit Commit Bot
Comment 5 2016-02-01 16:46:39 PST
Comment on attachment 269713 [details] Patch Clearing flags on attachment: 269713 Committed r195992: <http://trac.webkit.org/changeset/195992>
WebKit Commit Bot
Comment 6 2016-02-01 16:46:42 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.