[sr-dev] git:master:af9deb90: core: safety checks for swich statements with no case branch

Module: kamailio Branch: master Commit: af9deb9026a271aa3a1bdbb5685b63e3e78772bd URL: https://github.com/kamailio/kamailio/commit/af9deb9026a271aa3a1bdbb5685b63e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2015-05-14T13:33:50+02:00 core: safety checks for swich statements with no case branch --- Modified: switch.c --- Diff: https://github.com/kamailio/kamailio/commit/af9deb9026a271aa3a1bdbb5685b63e… Patch: https://github.com/kamailio/kamailio/commit/af9deb9026a271aa3a1bdbb5685b63e… --- diff --git a/switch.c b/switch.c index b4a6b75..6aeffc2 100644 --- a/switch.c +++ b/switch.c @@ -230,7 +230,9 @@ int fix_switch(struct action* t) LM_DBG("default only switch optimized away (BLOCK_T)\n"); return 0; } - LM_DBG("default only switch with side-effect...\n"); + LM_CRIT("default only switch with side-effect not expected at this point\n"); + ret=E_BUG; + goto error; }else{ LM_CRIT("empty switch not expected at this point\n"); ret=E_BUG; @@ -545,7 +547,9 @@ static int fix_match(struct action* t) LM_DBG("default only switch optimized away (BLOCK_T)\n"); return 0; } - LM_DBG("default only switch with side-effect...\n"); + LM_CRIT("default only switch with side-effect not expected at this point\n"); + ret=E_BUG; + goto error; }else{ LM_CRIT("empty switch not expected at this point\n"); ret=E_BUG;