Reviewed by Geoff Garen.
authortomernic <tomernic@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 31 Jan 2006 23:38:48 +0000 (23:38 +0000)
committertomernic <tomernic@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 31 Jan 2006 23:38:48 +0000 (23:38 +0000)
        * bindings/c/c_utility.cpp:
        (KJS::Bindings::convertUTF8ToUTF16):
Fixed an invalid assertion that UTF8Chars is not NULL.  It is valid for it to be NULL as long as
UTF8Length is 0.
This fixes an assertion failure on TOT at <http://www.musicindiaonline.com/p/x/tJO0OOBME9.As1NMvHdW/>,
where JavaScript is getting a NULL string back from some call on the Real Player plugin.

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

JavaScriptCore/ChangeLog
JavaScriptCore/bindings/c/c_utility.cpp

index a71599f452e2343335fbf3c809b9cb3e770a3981..70de30f0426ef18ff61ae3686a0c6e4172f19343 100644 (file)
@@ -1,3 +1,14 @@
+2006-01-31  Tim Omernick  <timo@apple.com>
+
+        Reviewed by Geoff Garen.
+
+        * bindings/c/c_utility.cpp:
+        (KJS::Bindings::convertUTF8ToUTF16):
+       Fixed an invalid assertion that UTF8Chars is not NULL.  It is valid for it to be NULL as long as
+       UTF8Length is 0.
+       This fixes an assertion failure on TOT at <http://www.musicindiaonline.com/p/x/tJO0OOBME9.As1NMvHdW/>,
+       where JavaScript is getting a NULL string back from some call on the Real Player plugin.
+
 2006-01-30  Anders Carlsson  <andersca@mac.com>
 
         Reviewed by Darin.
index f20dae4d7e6763c6efb7369dddcca016fee922dd..11a6d5bb302323c35c75e7ef0720a5855178c0de 100644 (file)
@@ -43,7 +43,7 @@ void convertNPStringToUTF16(const NPString *string, NPUTF16 **UTF16Chars, unsign
 // Requires free() of returned UTF16Chars.
 void convertUTF8ToUTF16(const NPUTF8 *UTF8Chars, int UTF8Length, NPUTF16 **UTF16Chars, unsigned int *UTF16Length)
 {
-    assert(UTF8Chars);
+    assert(UTF8Chars || UTF8Length == 0);
     
     if (UTF8Length == -1)
         UTF8Length = strlen(UTF8Chars);