app_voicemail: Fix json blob errors

When app_voicemail calls ast_test_suite_notify with the results of
a user keypress, it formats the keypress as '%c'.  If the user hung up
or some other error occurrs, the result of the keypress is a non
printable character.  This ultimately causes json_vpack_ex to think
it's being passed a non utf-8 string and return an error.

* Keypress results passed to ast_test_suite_notify are now checked with
  isprint() and a '?' is substituted if the check fails.

Change-Id: I78ee188916bbac840f3d03f40201b692347ea865

diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c
index 95d0e8dc058221aa49af5bb04838cc310044c7c5..234d8adabb38ac3e038578a6f388011eea112af1 100644 (file)
--- a/apps/app_voicemail.c
+++ b/apps/app_voicemail.c
@@ -7837,7 +7837,8 @@ static int get_folder2(struct ast_channel *chan, char *fn, int start)
                ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c", '#', '#');
                return '#';
        }
-       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c", res, res);
+       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c",
+               isprint(res) ? res : '?', isprint(res) ? res : '?');
        return res;
 }
 
@@ -8000,7 +8001,8 @@ static int vm_forwardoptions(struct ast_channel *chan, struct ast_vm_user *vmu,
                        if (retries > 3) {
                                cmd = '*'; /* Let's cancel this beast */
                        }
-                       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c", cmd, cmd);
+                       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c",
+                               isprint(cmd) ? cmd : '?', isprint(cmd) ? cmd : '?');
                }
        }
 
@@ -8228,7 +8230,8 @@ static int forward_message(struct ast_channel *chan, char *context, struct vm_st
                                                cmd = 't';
                                                done = 1;
                                        }
-                                       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c", cmd, cmd);
+                                       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c",
+                                               isprint(cmd) ? cmd : '?', isprint(cmd) ? cmd : '?');
                                }
                        }
                        if (cmd < 0 || cmd == 't')
@@ -8885,7 +8888,8 @@ static int play_message(struct ast_channel *chan, struct ast_vm_user *vmu, struc
                        ast_log(AST_LOG_WARNING, "Playback of message %s failed\n", vms->fn);
                        res = 0;
                }
-               ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c", res, res);
+               ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c",
+                       isprint(res) ? res : '?', isprint(res) ? res : '?');
        }
        DISPOSE(vms->curdir, vms->curmsg);
        return res;
@@ -10669,7 +10673,8 @@ static int vm_options(struct ast_channel *chan, struct ast_vm_user *vmu, struct
                        if (retries > 3) {
                                cmd = 't';
                        }
-                       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c", cmd, cmd);
+                       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c",
+                               isprint(cmd) ? cmd : '?', isprint(cmd) ? cmd : '?');
                }
        }
        if (cmd == 't')
@@ -10749,7 +10754,8 @@ static int vm_tempgreeting(struct ast_channel *chan, struct ast_vm_user *vmu, st
                                if (retries > 3) {
                                        cmd = 't';
                                }
-                               ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c", cmd, cmd);
+                               ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c",
+                                       isprint(cmd) ? cmd : '?', isprint(cmd) ? cmd : '?');
                        }
                }
                DISPOSE(prefile, -1);
@@ -11571,7 +11577,8 @@ static int vm_execmain(struct ast_channel *chan, const char *data)
        } else {
                cmd = vm_intro(chan, vmu, &vms);
        }
-       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c", cmd, cmd);
+       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c",
+               isprint(cmd) ? cmd : '?', isprint(cmd) ? cmd : '?');
 
        vms.repeats = 0;
        vms.starting = 1;
@@ -11591,7 +11598,8 @@ static int vm_execmain(struct ast_channel *chan, const char *data)
                                adsi_folders(chan, 0, "Change to folder...");
 
                        cmd = get_folder2(chan, "vm-changeto", 0);
-                       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c", cmd, cmd);
+                       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c",
+                               isprint(cmd) ? cmd : '?', isprint(cmd) ? cmd : '?');
                        if (cmd == '#') {
                                cmd = 0;
                        } else if (cmd > 0) {
@@ -11723,7 +11731,8 @@ static int vm_execmain(struct ast_channel *chan, const char *data)
                                        if (vms.repeats > 3) {
                                                cmd = 't';
                                        }
-                                       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c", cmd, cmd);
+                                       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c",
+                                               isprint(cmd) ? cmd : '?', isprint(cmd) ? cmd : '?');
                                }
                        }
                        if (cmd == 't') {
@@ -11901,7 +11910,8 @@ static int vm_execmain(struct ast_channel *chan, const char *data)
                        if (useadsi)
                                adsi_folders(chan, 1, "Save to folder...");
                        cmd = get_folder2(chan, "vm-savefolder", 1);
-                       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c", cmd, cmd);
+                       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c",
+                               isprint(cmd) ? cmd : '?', isprint(cmd) ? cmd : '?');
                        box = 0;        /* Shut up compiler */
                        if (cmd == '#') {
                                cmd = 0;
@@ -15132,7 +15142,8 @@ static int dialout(struct ast_channel *chan, struct ast_vm_user *vmu, char *num,
                                else
                                        cmd = 't';
                        }
-                       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c", cmd, cmd);
+                       ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c",
+                               isprint(cmd) ? cmd : '?', isprint(cmd) ? cmd : '?');
                }
                if (retries >= 3) {
                        return 0;
@@ -15297,7 +15308,8 @@ static int advanced_options(struct ast_channel *chan, struct ast_vm_user *vmu, s
                                                res = 't';
                                        }
                                }
-                               ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c", res, res);
+                               ast_test_suite_event_notify("USERPRESS", "Message: User pressed %c\r\nDTMF: %c",
+                                       isprint(res) ? res : '?', isprint(res) ? res : '?');
                                break;
 
                        }