[sr-dev] git:master:97189d75: app_ruby: detect if the exception is a script exit

Module: kamailio Branch: master Commit: 97189d7508a3946c2b3de49c178614356d816508 URL: https://github.com/kamailio/kamailio/commit/97189d7508a3946c2b3de49c1786143… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2019-03-22T05:41:07+01:00 app_ruby: detect if the exception is a script exit --- Modified: src/modules/app_ruby/app_ruby_api.c --- Diff: https://github.com/kamailio/kamailio/commit/97189d7508a3946c2b3de49c1786143… Patch: https://github.com/kamailio/kamailio/commit/97189d7508a3946c2b3de49c1786143… --- diff --git a/src/modules/app_ruby/app_ruby_api.c b/src/modules/app_ruby/app_ruby_api.c index ff944ca20e..7f4488a6e2 100644 --- a/src/modules/app_ruby/app_ruby_api.c +++ b/src/modules/app_ruby/app_ruby_api.c @@ -95,15 +95,20 @@ int ruby_sr_init_mod(void) return 0; } -static void app_ruby_print_last_exception() +static int app_ruby_print_last_exception() { VALUE rException, rExceptStr; rException = rb_errinfo(); /* get last exception */ rb_set_errinfo(Qnil); /* clear last exception */ rExceptStr = rb_funcall(rException, rb_intern("to_s"), 0, Qnil); - LM_ERR("exception: %s\n", StringValuePtr(rExceptStr)); - return; + if(RSTRING_LEN(rExceptStr)!=4 + || strncmp(RSTRING_PTR(rExceptStr), "exit", 4)!=0) { + LM_ERR("exception: %.*s\n", (int)RSTRING_LEN(rExceptStr), + RSTRING_PTR(rExceptStr)); + return 0; + } + return 1; } /** @@ -1073,10 +1078,11 @@ int app_ruby_run_ex(sip_msg_t *msg, char *func, char *p1, char *p2, _sr_R_env.msg = bmsg; if (rberr) { - app_ruby_print_last_exception(); - LM_ERR("ruby exception (%d) on callback for: %s (res type: %d)\n", - rberr, func, TYPE(rbres)); - return -1; + if(app_ruby_print_last_exception()==0) { + LM_ERR("ruby exception (%d) on callback for: %s (res type: %d)\n", + rberr, func, TYPE(rbres)); + return -1; + } } return 1;