[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:25:45 UTC 2009


The following commit has been merged in the debian/unstable branch:
commit 0f2bdea14a99d0c6df1417faf95498176ceeaf71
Author: darin <darin at 268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Sun Feb 23 23:24:15 2003 +0000

    Tests:
    
            Reviewed by Maciej.
    
            - added tests for URLs affected by bug 3179181, ones with a "?" or "#" character
              right after the host name, with no intervening "/"
            - turned off tests that aren't passing right now, so make check can be used pass/fail
              easy to turn tests back on by editing test.list
    
            * TestURLs.h: Added URLs "http://www.apple.com?email=darin@apple.com" and "http://my.site.com#fragment".
            * kde/kurl-test.chk: Added test results for the two new URLs.
    
            * WebFoundation-Misc/ifnsstringextensions-test.chk: Updated results for _web_splitAtNonDateCommas,
            which now splits only at commas followed by spaces.
    
            * test.list: Disabled ifnsthreadextensions-test, ifnsurlextensions-test, and ifurlparsing-test,
            since they aren't passing right now. We can turn them back on when we get them back in shape.
    
    WebCore:
    
            Reviewed by Maciej.
    
            - fixed 3179181 -- KURL won't accept URLs with a query part and no / separating host name from path
    
            * kwq/KWQKURL.mm: (KURL::parse): Allow any path segment end character to end the host part of a
            URL, rather than allowing only a "/".
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3690 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog-2003-10-25 b/WebCore/ChangeLog-2003-10-25
index 902850e..5ce62ce 100644
--- a/WebCore/ChangeLog-2003-10-25
+++ b/WebCore/ChangeLog-2003-10-25
@@ -1,3 +1,12 @@
+2003-02-23  Darin Adler  <darin at apple.com>
+
+        Reviewed by Maciej.
+
+        - fixed 3179181 -- KURL won't accept URLs with a query part and no / separating host name from path
+
+        * kwq/KWQKURL.mm: (KURL::parse): Allow any path segment end character to end the host part of a
+        URL, rather than allowing only a "/".
+
 2003-02-21  Maciej Stachowiak  <mjs at apple.com>
 
         Back out last change, mondo layout regressions.
diff --git a/WebCore/ChangeLog-2005-08-23 b/WebCore/ChangeLog-2005-08-23
index 902850e..5ce62ce 100644
--- a/WebCore/ChangeLog-2005-08-23
+++ b/WebCore/ChangeLog-2005-08-23
@@ -1,3 +1,12 @@
+2003-02-23  Darin Adler  <darin at apple.com>
+
+        Reviewed by Maciej.
+
+        - fixed 3179181 -- KURL won't accept URLs with a query part and no / separating host name from path
+
+        * kwq/KWQKURL.mm: (KURL::parse): Allow any path segment end character to end the host part of a
+        URL, rather than allowing only a "/".
+
 2003-02-21  Maciej Stachowiak  <mjs at apple.com>
 
         Back out last change, mondo layout regressions.
diff --git a/WebCore/kwq/KWQKURL.mm b/WebCore/kwq/KWQKURL.mm
index 59faf3d..95bcca2 100644
--- a/WebCore/kwq/KWQKURL.mm
+++ b/WebCore/kwq/KWQKURL.mm
@@ -868,7 +868,7 @@ void KURL::parse(const char *url, const QString *originalString)
 		passwordStart = passwordEnd = userEnd;
 	    }
 	    hostStart = passwordEnd + 1;
-	} else if (url[userEnd] == '\0' || url[userEnd] == '/' || url[userEnd] == '[') {
+	} else if (url[userEnd] == '\0' || url[userEnd] == '[' || IS_PATH_SEGMENT_END_CHAR(url[userEnd])) {
 	    // hit the end of the authority, must have been no user
 	    // or looks like an IPv6 hostname
 	    // either way, try to parse it as a hostname
@@ -916,7 +916,7 @@ void KURL::parse(const char *url, const QString *originalString)
 	    portStart = portEnd = hostEnd;
 	}
 
-	if (url[portEnd] != '\0' && url[portEnd] != '/') {
+	if (url[portEnd] != '\0' && !IS_PATH_SEGMENT_END_CHAR(url[portEnd])) {
 	    // invalid character
 	    m_isValid = false;
 	    urlString = url;

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list