diff options
author | russell <russell@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-05-10 13:22:15 +0000 |
---|---|---|
committer | russell <russell@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-05-10 13:22:15 +0000 |
commit | d99b677f3501944b7aaf82375ef62a88e5fa3933 (patch) | |
tree | 167bf7a9b6b330883dc9bf865672132658710ffd /res/res_clioriginate.c | |
parent | a6b2177d50659d049694ca79f6cbe00f54dd0a93 (diff) |
remove almost all of the checks of the result from ast_strdupa() or alloca().
As it turns out, all of these checks were useless, because alloca will never
return NULL.
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@26451 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'res/res_clioriginate.c')
-rw-r--r-- | res/res_clioriginate.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/res/res_clioriginate.c b/res/res_clioriginate.c index 1c02491d2..306975946 100644 --- a/res/res_clioriginate.c +++ b/res/res_clioriginate.c @@ -76,8 +76,7 @@ static int orig_app(int fd, const char *chan, const char *app, const char *appda if (ast_strlen_zero(app)) return RESULT_SHOWUSAGE; - if (!(chandata = ast_strdupa(chan))) - return RESULT_FAILURE; + chandata = ast_strdupa(chan); chantech = strsep(&chandata, "/"); if (!chandata) { @@ -98,8 +97,7 @@ static int orig_exten(int fd, const char *chan, const char *data) char *context = NULL; int reason = 0; - if (!(chandata = ast_strdupa(chan))) - return RESULT_FAILURE; + chandata = ast_strdupa(chan); chantech = strsep(&chandata, "/"); if (!chandata) { @@ -108,8 +106,7 @@ static int orig_exten(int fd, const char *chan, const char *data) } if (!ast_strlen_zero(data)) { - if (!(context = ast_strdupa(data))) - return RESULT_FAILURE; + context = ast_strdupa(data); exten = strsep(&context, "@"); } |