RESOLVED FIXED 57932
[GTK] DumpRenderTree: do not try to free NULL SoupURIs 
https://bugs.webkit.org/show_bug.cgi?id=57932
Summary [GTK] DumpRenderTree: do not try to free NULL SoupURIs
Sergio Villar Senin
Reported 2011-04-06 02:27:28 PDT
[GTK] DumpRenderTree: do not try to free NULL SoupURIs
Attachments
Patch (1.88 KB, patch)
2011-04-06 02:31 PDT, Sergio Villar Senin 		mrobinson: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Sergio Villar Senin
Comment 1 2011-04-06 02:31:11 PDT
Created attachment 88386 [details] Patch
Philippe Normand
Comment 2 2011-04-06 04:58:13 PDT
Comment on attachment 88386 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=88386&action=review Thanks for fixing this :) > Tools/DumpRenderTree/gtk/DumpRenderTree.cpp:1057 > + if (uri && SOUP_URI_VALID_FOR_HTTP(uri) && g_strcmp0(uri->host, "127.0.0.1") SOUP_URI_VALID_FOR_HTTP() already checks the uri, so no need for that extra check I think.
Martin Robinson
Comment 3 2011-04-06 06:53:09 PDT
Comment on attachment 88386 [details] Patch Nice fix. If Philippe is correct, please remove the first check.
Sergio Villar Senin
Comment 4 2011-04-06 10:32:49 PDT
Committed r83068: <http://trac.webkit.org/changeset/83068>
Note You need to log in before you can comment on or make changes to this bug.