SandboxExtension::createHandle raises an uncaught Objective-C exception if...
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 7 Mar 2013 07:25:20 +0000 (07:25 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 7 Mar 2013 07:25:20 +0000 (07:25 +0000)
        https://bugs.webkit.org/show_bug.cgi?id=111689

        Reviewed by Filip Pizlo.

        * Shared/mac/SandboxExtensionMac.mm: (WebKit::SandboxExtension::createHandle):
        Use a function from WebCore platform layer instead of Foundation one equivalent.
        We already do this elsewhere in this file.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@145042 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/Shared/mac/SandboxExtensionMac.mm

index d4ff8a8..062e726 100644 (file)
@@ -1,3 +1,14 @@
+2013-03-06  Alexey Proskuryakov  <ap@apple.com>
+
+        SandboxExtension::createHandle raises an uncaught Objective-C exception if path is empty
+        https://bugs.webkit.org/show_bug.cgi?id=111689
+
+        Reviewed by Filip Pizlo.
+
+        * Shared/mac/SandboxExtensionMac.mm: (WebKit::SandboxExtension::createHandle):
+        Use a function from WebCore platform layer instead of Foundation one equivalent.
+        We already do this elsewhere in this file.
+
 2013-03-06  Tim Horton  <timothy_horton@apple.com>
 
         [wk2] Page overlays shouldn't dirty the entire layer when flushing if the main frame can't scroll
index 13bd89f..6609ed6 100644 (file)
@@ -214,7 +214,8 @@ void SandboxExtension::createHandle(const String& path, Type type, Handle& handl
 {
     ASSERT(!handle.m_sandboxExtension);
 
-    CString standardizedPath = resolveSymlinksInPath([[(NSString *)path stringByStandardizingPath] fileSystemRepresentation]);
+    // FIXME: Do we need both resolveSymlinksInPath() and -stringByStandardizingPath?
+    CString standardizedPath = resolveSymlinksInPath(fileSystemRepresentation([(NSString *)path stringByStandardizingPath]));
     handle.m_sandboxExtension = WKSandboxExtensionCreate(standardizedPath.data(), wkSandboxExtensionType(type));
     if (!handle.m_sandboxExtension)
         WTFLogAlways("Could not create a sandbox extension for '%s'", path.utf8().data());