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 /funcs | |
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 'funcs')
-rw-r--r-- | funcs/func_cut.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/funcs/func_cut.c b/funcs/func_cut.c index a4bce49c1..4d3a85f3d 100644 --- a/funcs/func_cut.c +++ b/funcs/func_cut.c @@ -78,8 +78,7 @@ static int sort_internal(struct ast_channel *chan, char *data, char *buffer, siz if (!data) return ERROR_NOARG; - if (!(strings = ast_strdupa(data))) - return ERROR_NOMEM; + strings = ast_strdupa(data); for (ptrkey = strings; *ptrkey; ptrkey++) { if (*ptrkey == '|') @@ -87,8 +86,6 @@ static int sort_internal(struct ast_channel *chan, char *data, char *buffer, siz } sortable_keys = alloca(count * sizeof(struct sortable_keys)); - if (!sortable_keys) - return ERROR_NOMEM; memset(sortable_keys, 0, count * sizeof(struct sortable_keys)); @@ -132,8 +129,7 @@ static int cut_internal(struct ast_channel *chan, char *data, char *buffer, size memset(buffer, 0, buflen); - if (!(parse = ast_strdupa(data))) - return ERROR_NOMEM; + parse = ast_strdupa(data); AST_STANDARD_APP_ARGS(args, parse); |