Merged revisions 70899 via svnmerge from
authorJoshua Colp <jcolp@digium.com>
Thu, 21 Jun 2007 21:41:45 +0000 (21:41 +0000)
committerJoshua Colp <jcolp@digium.com>
Thu, 21 Jun 2007 21:41:45 +0000 (21:41 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.4

................
r70899 | file | 2007-06-21 17:40:19 -0400 (Thu, 21 Jun 2007) | 10 lines

Merged revisions 70898 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r70898 | file | 2007-06-21 17:37:55 -0400 (Thu, 21 Jun 2007) | 2 lines

Don't explode if the gain option is specified without a value. (issue #9274 reported by mfarver)

........

................

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@70900 65c4cc65-6c06-0410-ace0-fbb531ad65f3

apps/app_voicemail.c

index 167f108..b68337c 100644 (file)
@@ -6554,7 +6554,7 @@ static int vm_execmain(struct ast_channel *chan, void *data)
                        }
                        if (ast_test_flag(&flags, OPT_RECORDGAIN)) {
                                int gain;
-                               if (opts[OPT_ARG_RECORDGAIN]) {
+                               if (!ast_strlen_zero(opts[OPT_ARG_RECORDGAIN])) {
                                        if (sscanf(opts[OPT_ARG_RECORDGAIN], "%d", &gain) != 1) {
                                                ast_log(LOG_WARNING, "Invalid value '%s' provided for record gain option\n", opts[OPT_ARG_RECORDGAIN]);
                                                ast_module_user_remove(u);