diff options
| author | Russell Bryant <russell@russellbryant.com> | 2006-05-10 13:22:15 +0000 |
|---|---|---|
| committer | Russell Bryant <russell@russellbryant.com> | 2006-05-10 13:22:15 +0000 |
| commit | 04ecb29d03ed942e53917974b8d8580b5341b42e (patch) | |
| tree | 167bf7a9b6b330883dc9bf865672132658710ffd /funcs/func_cut.c | |
| parent | 8e897e1a531bb1d63a7c53bd367e97e8b2fa0952 (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: https://origsvn.digium.com/svn/asterisk/trunk@26451 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'funcs/func_cut.c')
| -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); |