diff options
author | Russell Bryant <russell@russellbryant.com> | 2005-10-26 19:48:14 +0000 |
---|---|---|
committer | Russell Bryant <russell@russellbryant.com> | 2005-10-26 19:48:14 +0000 |
commit | 5187de9ab6e5abc33ee6de4c4b16dbd3e3a5a7a7 (patch) | |
tree | cc336d91bb85ce45338470c9de5c8fd8717c9c73 /apps/app_osplookup.c | |
parent | dbc5c06c94d4472730e25c77f9b4e3d59db6b6ee (diff) |
remove unnecessary checks before calls to ast_strlen_zero
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@6864 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'apps/app_osplookup.c')
-rwxr-xr-x | apps/app_osplookup.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/apps/app_osplookup.c b/apps/app_osplookup.c index 6a0bc6aec..4cb8f2b04 100755 --- a/apps/app_osplookup.c +++ b/apps/app_osplookup.c @@ -110,7 +110,7 @@ static int osplookup_exec(struct ast_channel *chan, void *data) char *provider, *opts=NULL; struct ast_osp_result result; - if (!data || ast_strlen_zero(data)) { + if (ast_strlen_zero(data)) { ast_log(LOG_WARNING, "OSPLookup requires an argument (extension)\n"); return -1; } @@ -169,7 +169,7 @@ static int ospnext_exec(struct ast_channel *chan, void *data) int cause; struct ast_osp_result result; - if (!data || ast_strlen_zero(data)) { + if (ast_strlen_zero(data)) { ast_log(LOG_WARNING, "OSPNext should have an argument (cause)\n"); return -1; } @@ -179,7 +179,7 @@ static int ospnext_exec(struct ast_channel *chan, void *data) cause = str2cause((char *)data); temp = pbx_builtin_getvar_helper(chan, "OSPHANDLE"); result.handle = -1; - if (temp && strlen(temp) && (sscanf(temp, "%d", &result.handle) == 1) && (result.handle > -1)) { + if (!ast_strlen_zero(temp) && (sscanf(temp, "%d", &result.handle) == 1) && (result.handle > -1)) { if ((res = ast_osp_next(&result, cause)) > 0) { char tmp[80]; snprintf(tmp, sizeof(tmp), "%d", result.handle); @@ -217,7 +217,7 @@ static int ospfinished_exec(struct ast_channel *chan, void *data) time_t start=0, duration=0; struct ast_osp_result result; - if (!data || ast_strlen_zero(data)) { + if (ast_strlen_zero(data)) { ast_log(LOG_WARNING, "OSPFinish should have an argument (cause)\n"); return -1; } @@ -236,7 +236,7 @@ static int ospfinished_exec(struct ast_channel *chan, void *data) cause = str2cause((char *)data); temp = pbx_builtin_getvar_helper(chan, "OSPHANDLE"); result.handle = -1; - if (temp && strlen(temp) && (sscanf(temp, "%d", &result.handle) == 1) && (result.handle > -1)) { + if (!ast_strlen_zero(temp) && (sscanf(temp, "%d", &result.handle) == 1) && (result.handle > -1)) { if (!ast_osp_terminate(result.handle, cause, start, duration)) { pbx_builtin_setvar_helper(chan, "_OSPHANDLE", ""); res = 1; |