Changeset 24884 in webkit for trunk/JavaScriptCore/API/JSBase.cpp

Timestamp:

Aug 6, 2007, 12:00:46 PM (18 years ago)

Author:

darin

Message:

Reviewed by Anders.

• fix <rdar://problem/5387589> 9A514: Quartz Composer crash on launch in KJS::jsString

• API/JSBase.cpp: (JSEvaluateScript): Turn NULL for sourceURL into UString::null(), just as JSObjectMakeFunction already does. (JSCheckScriptSyntax): Ditto.

File:

• trunk/JavaScriptCore/API/JSBase.cpp (modified) (2 diffs)

trunk/JavaScriptCore/API/JSBase.cpp

@@ -43 +43 @@
     JSObject* jsThisObject = toJS(thisObject);
     UString::Rep* scriptRep = toJS(script);
-    UString::Rep* sourceURLRep = toJS(sourceURL);
+    UString::Rep* sourceURLRep = sourceURL ? toJS(sourceURL) : &UString::Rep::null;
     // Interpreter::evaluate sets "this" to the global object if it is NULL
     Completion completion = exec->dynamicInterpreter()->evaluate(UString(sourceURLRep), startingLineNumber, UString(scriptRep), jsThisObject);
@@ -66 +66 @@
     ExecState* exec = toJS(ctx);
     UString::Rep* scriptRep = toJS(script);
-    UString::Rep* sourceURLRep = toJS(sourceURL);
+    UString::Rep* sourceURLRep = sourceURL ? toJS(sourceURL) : &UString::Rep::null;
     Completion completion = exec->dynamicInterpreter()->checkSyntax(UString(sourceURLRep), startingLineNumber, UString(scriptRep));
     if (completion.complType() == Throw) {