Title: [116200] trunk/Source/WebKit2
Revision
116200
Author
[email protected]
Date
2012-05-04 18:17:41 -0700 (Fri, 04 May 2012)

Log Message

Fix a leak in WebProcess when it is used to launch the UI process.

Rubber-stamped by Anders Carlsson.

* WebProcess/mac/WebProcessMainMac.mm:
(WebKit::WebProcessMain): Destory the attributes and file actions after
spawning the subprocess.

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (116199 => 116200)


--- trunk/Source/WebKit2/ChangeLog	2012-05-05 01:12:54 UTC (rev 116199)
+++ trunk/Source/WebKit2/ChangeLog	2012-05-05 01:17:41 UTC (rev 116200)
@@ -1,3 +1,13 @@
+2012-05-04  Mark Rowe  <[email protected]>
+
+        Fix a leak in WebProcess when it is used to launch the UI process.
+
+        Rubber-stamped by Anders Carlsson.
+
+        * WebProcess/mac/WebProcessMainMac.mm:
+        (WebKit::WebProcessMain): Destory the attributes and file actions after
+        spawning the subprocess.
+
 2012-05-02  Jer Noble  <[email protected]>
 
         Flash of white when exiting full screen HTML5 video

Modified: trunk/Source/WebKit2/WebProcess/mac/WebProcessMainMac.mm (116199 => 116200)


--- trunk/Source/WebKit2/WebProcess/mac/WebProcessMainMac.mm	2012-05-05 01:12:54 UTC (rev 116199)
+++ trunk/Source/WebKit2/WebProcess/mac/WebProcessMainMac.mm	2012-05-05 01:17:41 UTC (rev 116200)
@@ -118,6 +118,10 @@
         posix_spawnattr_setflags(&attributes, POSIX_SPAWN_CLOEXEC_DEFAULT | POSIX_SPAWN_SETPGROUP);
 
         int spawnResult = posix_spawn(0, command.data(), &fileActions, &attributes, const_cast<char**>(args), environmentVariables.environmentPointer());
+
+        posix_spawnattr_destroy(&attributes);
+        posix_spawn_file_actions_destroy(&fileActions);
+
         if (spawnResult)
             return 2;
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to