Reviewed by Maciej.
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 9 Mar 2005 19:00:31 +0000 (19:00 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 9 Mar 2005 19:00:31 +0000 (19:00 +0000)
        <rdar://problem/4040388> REGRESSION (172-173): nonrepro crash in -[NSString(WebNSURLExtras) _web_isUserVisibleURL]

        * Misc.subproj/WebNSURLExtras.m:
        (-[NSString _web_isUserVisibleURL]): Fixed some pointer expressions that didn't include the index in the expression.
        (readIDNScriptWhiteListFile): Removed NSLog statements in here since we decided they aren't useful and they will
        write some messages on Tiger.

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

WebKit/ChangeLog
WebKit/Misc.subproj/WebNSURLExtras.m

index 0fb1c05c8958124b63f0f89d26f0905cd8658121..a990e76f5867901211a7c471aaae6f1e35fa1782 100644 (file)
@@ -1,3 +1,14 @@
+2005-03-09  Darin Adler  <darin@apple.com>
+
+        Reviewed by Maciej.
+
+        <rdar://problem/4040388> REGRESSION (172-173): nonrepro crash in -[NSString(WebNSURLExtras) _web_isUserVisibleURL]
+
+        * Misc.subproj/WebNSURLExtras.m:
+        (-[NSString _web_isUserVisibleURL]): Fixed some pointer expressions that didn't include the index in the expression.
+        (readIDNScriptWhiteListFile): Removed NSLog statements in here since we decided they aren't useful and they will
+        write some messages on Tiger.
+
 2005-03-09  Darin Adler  <darin@apple.com>
 
         * DOM.subproj/DOMPrivate.h: Checked in file copied from WebCore.
index bfe83269213498cecf1bc923d6ca76230c3a1266..0598ee46917c398da4ee9f6da6a74ea555c7c779 100644 (file)
@@ -736,7 +736,7 @@ static NSString *mapHostNames(NSString *string, BOOL encode)
             i += 2;
         } else {
             // Check for "xn--" in an efficient, non-case-sensitive, way.
-            if (c == '-' && i >= 3 && (p[-3] | 0x20) == 'x' && (p[-2] | 0x20) == 'n' && p[-1] == '-') {
+            if (c == '-' && i >= 3 && (p[i - 3] | 0x20) == 'x' && (p[i - 2] | 0x20) == 'n' && p[i - 1] == '-') {
                 valid = NO;
                 break;
             }
@@ -805,14 +805,9 @@ static BOOL readIDNScriptWhiteListFile(NSString *filename)
         if (result == 1) {
             // Got a word, map to script code and put it into the array.
             int32_t script = u_getPropertyValueEnum(UCHAR_SCRIPT, word);
-            if (script == UCHAR_INVALID_CODE) {
-                NSLog(@"%@: unknown script code: %s", filename, word);
-            } else if (script >= 0 && script < USCRIPT_CODE_LIMIT) {
+            if (script >= 0 && script < USCRIPT_CODE_LIMIT) {
                 size_t index = script / 32;
                 uint32_t mask = 1 << (script % 32);
-                if (IDNScriptWhiteList[index] & mask) {
-                    NSLog(@"%@: script code %s is listed twice\n", filename, word);
-                }
                 IDNScriptWhiteList[index] |= mask;
             }
         }