Reviewed by Darin.
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 11 Sep 2004 08:22:27 +0000 (08:22 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 11 Sep 2004 08:22:27 +0000 (08:22 +0000)
<rdar://problem/3787208> can't log in to SAP SDN site (JS parse error)

        * kwq/KWQKURL.mm:
        (KURL::decode_string): Check for find result >= 0 instead of >0,
to handle case of first character in a string being encoded.

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

WebCore/ChangeLog-2005-08-23
WebCore/kwq/KWQKURL.mm

index 8b16a91..11c49fa 100644 (file)
@@ -1,3 +1,13 @@
+2004-09-11  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Darin.
+
+       <rdar://problem/3787208> can't log in to SAP SDN site (JS parse error)
+
+        * kwq/KWQKURL.mm:
+        (KURL::decode_string): Check for find result >= 0 instead of >0,
+       to handle case of first character in a string being encoded.
+
 2004-09-09  Chris Blumenberg  <cblu@apple.com>
 
        Support for: <rdar://problem/3795485> debug menu item to enable RSS animation on first layout
index 2a2211a..9accd36 100644 (file)
@@ -905,7 +905,7 @@ QString KURL::decode_string(const QString &urlString, const QTextCodec *codec)
     int decodedPosition = 0;
     int searchPosition = 0;
     int encodedRunPosition;
-    while ((encodedRunPosition = urlString.find('%', searchPosition)) > 0) {
+    while ((encodedRunPosition = urlString.find('%', searchPosition)) >= 0) {
         // Find the sequence of %-escape codes.
         int encodedRunEnd = encodedRunPosition;
         while (length - encodedRunEnd >= 3