[Asterisk-code-review] Fixing app voicemail-VoiceMailPlayMsg (asterisk[13])

[Ryan Rittgarn]

Ryan Rittgarn has uploaded a new change for review. ( https://gerrit.asterisk.org/4817 )

Change subject: Fixing app_voicemail-VoiceMailPlayMsg
......................................................................

Fixing app_voicemail-VoiceMailPlayMsg

Removed extraneous make_file call, added
necessary call to RETRIEVE and cleaned up
after ourselves with DISPOSE

Change-Id: I1e122dd53c0f3d7faa10f3c2b7e7e76a47d51b8c
---
M apps/app_voicemail.c
1 file changed, 3 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/17/4817/1

diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c
index bee684e..1e9bec3 100644
--- a/apps/app_voicemail.c
+++ b/apps/app_voicemail.c
@@ -11127,7 +11127,6 @@
 	/* Found the msg, so play it back */
 
 	make_file(vms->fn, sizeof(vms->fn), vms->curdir, vms->curmsg);
-	make_file(vms->fn, sizeof(vms->fn), vms->curdir, vms->curmsg);
 
 #ifdef IMAP_STORAGE
 	/*IMAP storage stores any prepended message from a forward
@@ -11137,6 +11136,8 @@
 		wait_file(chan, vms, vms->introfn);
 	}
 #endif
+	RETRIEVE(vms->curdir,vms->curmsg,vmu->mailbox, vmu->context);
+
 	if ((wait_file(chan, vms, vms->fn)) < 0) {
 		ast_log(AST_LOG_WARNING, "Playback of message %s failed\n", vms->fn);
 	} else {
@@ -11148,7 +11149,7 @@
 		ast_mutex_unlock(&vms->lock);
 #endif
 	}
-
+	DISPOSE(vms->curdir, vms->curmsg);
 	return 0;
 }
 

-- 
To view, visit https://gerrit.asterisk.org/4817
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1e122dd53c0f3d7faa10f3c2b7e7e76a47d51b8c
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Ryan Rittgarn <rrittgarn at techpro.com>