diff options
| author | mmichelson <mmichelson@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-03-24 22:42:00 +0000 |
|---|---|---|
| committer | mmichelson <mmichelson@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-03-24 22:42:00 +0000 |
| commit | c59d8265ee20c3de2f72e1f1beaa9140abbd5ef4 (patch) | |
| tree | 019140098d71c78229e4180fe594c70a5ed1b3c1 /apps/app_senddtmf.c | |
| parent | 8f9141628149b784fe31272859801c9308eae0c9 (diff) | |
Merged revisions 184079 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk
................
r184079 | mmichelson | 2009-03-24 17:40:39 -0500 (Tue, 24 Mar 2009) | 15 lines
Merged revisions 184078 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r184078 | mmichelson | 2009-03-24 17:34:45 -0500 (Tue, 24 Mar 2009) | 9 lines
Change NULL pointer check to be ast_strlen_zero.
The 'digit' variable is guaranteed to be non-NULL, so the if
statement could never evaluate true. Changing to ast_strlen_zero
makes the logic correct.
This was found while reviewing ast_channel_ao2 code review.
........
................
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.1@184081 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'apps/app_senddtmf.c')
| -rw-r--r-- | apps/app_senddtmf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/apps/app_senddtmf.c b/apps/app_senddtmf.c index 9863ce012..5a64ab1b0 100644 --- a/apps/app_senddtmf.c +++ b/apps/app_senddtmf.c @@ -92,7 +92,7 @@ static int manager_play_dtmf(struct mansession *s, const struct message *m) astman_send_error(s, m, "Channel not specified"); return 0; } - if (!digit) { + if (ast_strlen_zero(digit)) { astman_send_error(s, m, "No digit specified"); ast_channel_unlock(chan); return 0; |