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 /apps/app_realtime.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 'apps/app_realtime.c')
| -rw-r--r-- | apps/app_realtime.c | 34 |
1 files changed, 16 insertions, 18 deletions
diff --git a/apps/app_realtime.c b/apps/app_realtime.c index 49aa31cc3..a1bd140a8 100644 --- a/apps/app_realtime.c +++ b/apps/app_realtime.c @@ -149,16 +149,15 @@ static int realtime_update_exec(struct ast_channel *chan, void *data) LOCAL_USER_ADD(u); - if ((family = ast_strdupa(data))) { - if ((colmatch = strchr(family,'|'))) { - crop_data(colmatch); - if ((value = strchr(colmatch,'|'))) { - crop_data(value); - if ((newcol = strchr(value,'|'))) { - crop_data(newcol); - if ((newval = strchr(newcol,'|'))) - crop_data(newval); - } + family = ast_strdupa(data); + if ((colmatch = strchr(family,'|'))) { + crop_data(colmatch); + if ((value = strchr(colmatch,'|'))) { + crop_data(value); + if ((newcol = strchr(value,'|'))) { + crop_data(newcol); + if ((newval = strchr(newcol,'|'))) + crop_data(newval); } } } @@ -196,14 +195,13 @@ static int realtime_exec(struct ast_channel *chan, void *data) LOCAL_USER_ADD(u); - if ((family = ast_strdupa(data))) { - if ((colmatch = strchr(family,'|'))) { - crop_data(colmatch); - if ((value = strchr(colmatch,'|'))) { - crop_data(value); - if ((prefix = strchr(value,'|'))) - crop_data(prefix); - } + family = ast_strdupa(data); + if ((colmatch = strchr(family,'|'))) { + crop_data(colmatch); + if ((value = strchr(colmatch,'|'))) { + crop_data(value); + if ((prefix = strchr(value,'|'))) + crop_data(prefix); } } if (! (family && value && colmatch) ) { |