Fix potential for crash after ReferenceError during function declarat…

…ion (fix #2457)

This time do it without breaking everything

ChangeLog

@@ -17,6 +17,7 @@
             Fix issue with g.wrapString when running on flash-based strings
             Fix lock leak when using flat/flash/native strings as object indices
             Fix g.wrapString lockup if wrap width is less than the character width
+            Fix potential for crash after ReferenceError during function declaration (fix #2457)
 
      2v20 : Ensure String.charCodeAt returns NaN for out of bounds chars
             Bangle.js2: When rendering overlays, *do not* use the current FG/BG color for 1 bit overlays

src/jsparse.c

@@ -947,7 +947,8 @@ NO_INLINE JsVar *jspeFunctionCall(JsVar *function, JsVar *functionName, JsVar *t
 
     return returnVar;
   } else if (isParsing) { // ---------------------------------- function, but not executing - just parse args and be done
-    jspeParseFunctionCallBrackets();
+    if (jspCheckStackPosition()) // check for stack overflow
+      jspeParseFunctionCallBrackets();
     /* Do not return function, as it will be unlocked! */
     return 0;
   } else return 0;
@@ -1319,7 +1320,6 @@ NO_INLINE JsVar *jspeFactorFunctionCall() {
   while ((lex->tk=='(' || (isConstructor && JSP_SHOULD_EXECUTE)) && !jspIsInterrupted()) {
     JsVar *funcName = a;
     JsVar *func = jsvSkipName(funcName);
-
     /* The constructor function doesn't change parsing, so if we're
      * not executing, just short-cut it. */
     if (isConstructor && JSP_SHOULD_EXECUTE) {