[Freeswitch-dev] Unreachable code in mod_spidermonkey.c (event_chat_execute)

[Niels Thomsen]

There's a bit of unreachable code in mod_spidermonkey which means that
the event_chat_execute function becomes a no-op.

 

This may be intentional, of course, but it doesn't look like it is...

 

static JSBool event_chat_execute(JSContext * cx, JSObject * obj, uintN
argc, jsval * argv, jsval * rval)

{

                struct event_obj *eo = JS_GetPrivate(cx, obj);

 

                if (eo) {

                                if (argc > 0) {

                                                char *app =
JS_GetStringBytes(JS_ValueToString(cx, argv[0]));

                                                char *arg = NULL;

 

                                                if (argc > 1) {

                                                                arg =
JS_GetStringBytes(JS_ValueToString(cx, argv[1]));

                                                }

 

                                                goto end;

 

//-- this code is never executed                                

 
switch_core_execute_chat_app(eo->event, app, arg);

 

                                                *rval =
BOOLEAN_TO_JSVAL(JS_TRUE);

                                                return JS_TRUE;

                                }

                }

 

end:

 

                *rval = BOOLEAN_TO_JSVAL(JS_FALSE);

                return JS_FALSE;

}

 

- Niels

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.freeswitch.org/pipermail/freeswitch-dev/attachments/20121001/b5c2b18a/attachment-0001.html