From 4168fa3466c5eacdf892d08f868e53b417ac1df0 Mon Sep 17 00:00:00 2001 From: Naveen Albert Date: Mon, 14 Nov 2022 13:12:19 +0000 Subject: [PATCH] app_voicemail_odbc: Fix string overflow warning. Fixes a negative offset warning by initializing the buffer to empty. Additionally, although it doesn't currently complain about it, the size of a buffer is increased to accomodate the maximum size contents it could have. ASTERISK-30240 #close Change-Id: I8eecedf14d3f2a75864797f802277cac89a32877 --- apps/app_voicemail.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c index 1aca59b766..88dc342ee1 100644 --- a/apps/app_voicemail.c +++ b/apps/app_voicemail.c @@ -4484,15 +4484,16 @@ static void rename_file(char *sdir, int smsg, char *mailboxuser, char *mailboxco */ static int remove_file(char *dir, int msgnum) { - char fn[PATH_MAX]; - char full_fn[PATH_MAX]; + char fn[PATH_MAX] = ""; + char full_fn[PATH_MAX + 4]; /* Plus .txt */ char msgnums[80]; if (msgnum > -1) { snprintf(msgnums, sizeof(msgnums), "%d", msgnum); make_file(fn, sizeof(fn), dir, msgnum); - } else + } else { ast_copy_string(fn, dir, sizeof(fn)); + } ast_filedelete(fn, NULL); snprintf(full_fn, sizeof(full_fn), "%s.txt", fn); unlink(full_fn);