[SCM] WebKit Debian packaging branch, debian/unstable, updated. debian/1.1.15-1-40151-g37bb677

darin darin at 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Sat Sep 26 07:19:04 UTC 2009


The following commit has been merged in the debian/unstable branch:
commit 7b10f2ffd5f29c4f5409be1d8ae27f150a43b17b
Author: darin <darin at 268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Fri Jan 10 01:33:44 2003 +0000

            Reviewed by Chris.
    
    	- second pass on 3143332 - we still need a test case to be sure this is right
    
            * Plugins.subproj/WebBaseNetscapePluginStream.m:
            (-[WebBaseNetscapePluginStream finishedLoadingWithData:]):
    	Check for "/". Also report the failure instead of just hanging on forever.
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3289 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebKit/ChangeLog b/WebKit/ChangeLog
index 598d15c..75fe367 100644
--- a/WebKit/ChangeLog
+++ b/WebKit/ChangeLog
@@ -1,5 +1,15 @@
 2003-01-09  Darin Adler  <darin at apple.com>
 
+        Reviewed by Chris.
+
+	- second pass on 3143332 - we still need a test case to be sure this is right
+
+        * Plugins.subproj/WebBaseNetscapePluginStream.m:
+        (-[WebBaseNetscapePluginStream finishedLoadingWithData:]):
+	Check for "/". Also report the failure instead of just hanging on forever.
+
+2003-01-09  Darin Adler  <darin at apple.com>
+
         Reviewed by Richard.
         
         - fixed 3143332 -- if path returned is empty string (or "." or "..") it will trash the /tmp symlink
diff --git a/WebKit/Plugins.subproj/WebBaseNetscapePluginStream.m b/WebKit/Plugins.subproj/WebBaseNetscapePluginStream.m
index fc080ca..9029231 100644
--- a/WebKit/Plugins.subproj/WebBaseNetscapePluginStream.m
+++ b/WebKit/Plugins.subproj/WebBaseNetscapePluginStream.m
@@ -136,7 +136,14 @@
     }
     
     NSString *filename = [[URL path] lastPathComponent];
-    if ([filename length] == 0 || [filename isEqualToString:@"."] || [filename isEqualToString:@".."]) {
+    
+    // It's very important to not do the below calls, like "removeFileAtPath:", on empty paths.
+    // That's the way you lose your "/tmp" directory.
+    if ([filename length] == 0
+            || [filename isEqualToString:@"."]
+            || [filename isEqualToString:@".."]
+            || [filename isEqualToString:@"/"]) {
+        [self destroyStreamWithReason:NPRES_NETWORK_ERR];
         return;
     }
     

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list