Title: [125515] trunk/Tools
Revision
125515
Author
[email protected]
Date
2012-08-14 00:31:34 -0700 (Tue, 14 Aug 2012)

Log Message

[GTK] String returned by g_path_get_dirname() must be freed
https://bugs.webkit.org/show_bug.cgi?id=93885

Patch by Sudarsana Nagineni <[email protected]> on 2012-08-14
Reviewed by Philippe Normand.

Fix a memory leak in GTK's DRT code.

* DumpRenderTree/gtk/DumpRenderTree.cpp:
(pathFromSoupURI): Free the string returned by g_path_get_dirname().

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (125514 => 125515)


--- trunk/Tools/ChangeLog	2012-08-14 07:25:47 UTC (rev 125514)
+++ trunk/Tools/ChangeLog	2012-08-14 07:31:34 UTC (rev 125515)
@@ -1,3 +1,15 @@
+2012-08-14  Sudarsana Nagineni  <[email protected]>
+
+        [GTK] String returned by g_path_get_dirname() must be freed
+        https://bugs.webkit.org/show_bug.cgi?id=93885
+
+        Reviewed by Philippe Normand.
+
+        Fix a memory leak in GTK's DRT code.
+
+        * DumpRenderTree/gtk/DumpRenderTree.cpp:
+        (pathFromSoupURI): Free the string returned by g_path_get_dirname().
+
 2012-08-14  Zan Dobersek  <[email protected]>
 
         Unreviewed GTK build fix after r125508.

Modified: trunk/Tools/DumpRenderTree/gtk/DumpRenderTree.cpp (125514 => 125515)


--- trunk/Tools/DumpRenderTree/gtk/DumpRenderTree.cpp	2012-08-14 07:25:47 UTC (rev 125514)
+++ trunk/Tools/DumpRenderTree/gtk/DumpRenderTree.cpp	2012-08-14 07:31:34 UTC (rev 125515)
@@ -1126,7 +1126,8 @@
         return CString(uriString.get());
     }
 
-    GOwnPtr<gchar> pathDirname(g_path_get_basename(g_path_get_dirname(uri->path)));
+    GOwnPtr<gchar> parentPath(g_path_get_dirname(uri->path));
+    GOwnPtr<gchar> pathDirname(g_path_get_basename(parentPath.get()));
     GOwnPtr<gchar> pathBasename(g_path_get_basename(uri->path));
     GOwnPtr<gchar> urlPath(g_strdup_printf("%s/%s", pathDirname.get(), pathBasename.get()));
     return CString(urlPath.get());
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to