diff options
author | Russell Bryant <russell@russellbryant.com> | 2005-05-26 15:50:50 +0000 |
---|---|---|
committer | Russell Bryant <russell@russellbryant.com> | 2005-05-26 15:50:50 +0000 |
commit | 2cd9d4c67669c0cebb526039d3e982d81dfe69b9 (patch) | |
tree | 351903ef14a0ad8e6f8a787e97793d56016618d3 /pbx.c | |
parent | a6f2f031b35cd953b47cd7712d795db465c543ad (diff) |
fix return value for ExecIfTime when time is outside of the specified range (bug #4380)
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@5773 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'pbx.c')
-rwxr-xr-x | pbx.c | 10 |
1 files changed, 6 insertions, 4 deletions
@@ -5443,7 +5443,12 @@ static int pbx_builtin_execiftime(struct ast_channel *chan, void *data) ptr2 = ptr1; /* Separate the Application data ptr1 is the time spec ptr2 is the app|data*/ strsep(&ptr2,"?"); - if (ast_build_timing(&timing, ptr1) && ast_check_timing(&timing)) { + if(!(res = ast_build_timing(&timing, ptr1))) { + ast_log(LOG_WARNING, "Invalid Time Spec: %s\nCorrect usage: %s\n", ptr1, usage); + res = -1; + } + + if (!res && ast_check_timing(&timing)) { if (ptr2) { /* ptr2 is now the app name we're done with ptr1 now so recycle it and use it to point to the app args*/ @@ -5461,9 +5466,6 @@ static int pbx_builtin_execiftime(struct ast_channel *chan, void *data) } else { ast_log(LOG_WARNING, "%s\n", usage); } - } else { - ast_log(LOG_WARNING, "Invalid Time Spec: %s\nCorrect usage: %s\n", ptr1, usage); - res = -1; } } else { ast_log(LOG_ERROR, "Memory Error!\n"); |