Title: [111464] trunk/Source/WebKit2
Revision
111464
Author
[email protected]
Date
2012-03-20 16:33:23 -0700 (Tue, 20 Mar 2012)

Log Message

        Address review comment for WebProcess side, too.

        * WebProcess/mac/WebProcessMac.mm: (WebKit::initializeSandbox): There is no need for
        path to be in static storage, setenv will copy the value.

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (111463 => 111464)


--- trunk/Source/WebKit2/ChangeLog	2012-03-20 23:30:39 UTC (rev 111463)
+++ trunk/Source/WebKit2/ChangeLog	2012-03-20 23:33:23 UTC (rev 111464)
@@ -1,5 +1,12 @@
 2012-03-20  Alexey Proskuryakov  <[email protected]>
 
+        Address review comment for WebProcess side, too.
+
+        * WebProcess/mac/WebProcessMac.mm: (WebKit::initializeSandbox): There is no need for
+        path to be in static storage, setenv will copy the value.
+
+2012-03-20  Alexey Proskuryakov  <[email protected]>
+
         Sandboxed PluginProcess should use private temporary and cache directories
         https://bugs.webkit.org/show_bug.cgi?id=81702
         <rdar://problem/10792047>

Modified: trunk/Source/WebKit2/WebProcess/mac/WebProcessMac.mm (111463 => 111464)


--- trunk/Source/WebKit2/WebProcess/mac/WebProcessMac.mm	2012-03-20 23:30:39 UTC (rev 111463)
+++ trunk/Source/WebKit2/WebProcess/mac/WebProcessMac.mm	2012-03-20 23:33:23 UTC (rev 111464)
@@ -183,7 +183,7 @@
     // Use private temporary and cache directories.
     String systemDirectorySuffix = "com.apple.WebProcess+" + parameters.uiProcessBundleIdentifier;
     setenv("DIRHELPER_USER_DIR_SUFFIX", fileSystemRepresentation(systemDirectorySuffix).data(), 0);
-    static char temporaryDirectory[PATH_MAX];
+    char temporaryDirectory[PATH_MAX];
     if (!confstr(_CS_DARWIN_USER_TEMP_DIR, temporaryDirectory, sizeof(temporaryDirectory))) {
         fprintf(stderr, "WebProcess: couldn't retrieve private temporary directory path: %d\n", errno);
         exit(EX_NOPERM);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to