issue #5718

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

ChangeLog

@@ -1,5 +1,7 @@
 2005-11-11  Kevin P. Fleming  <kpfleming@digium.com>
 
+	* apps/app_voicemail.c (forward_message): correct bugs in message forwarding (issue #5718)
+
 	* apps/app_dial.c (wait_for_answer): correct flag copying for automon feature (issue #5720)
 
 	* channels/chan_iax2.c: correct comment

apps/app_voicemail.c

@@ -3479,9 +3479,15 @@ static int forward_message(struct ast_channel *chan, char *context, char *dir, i
 				ast_log(LOG_DEBUG, "%s", sys);
 				ast_safe_system(sys);
 		
-				if ( (res = count_messages(receiver, todir)) )
-					break;
+				res = count_messages(receiver, todir);
+
+				if ( (res == ERROR_LOCK_PATH) || (res < 0) ) {
+					if (res == ERROR_LOCK_PATH)
+						ast_log(LOG_WARNING, "Unable to lock the directory %s to forward the requested vmail msg!\n", todir);
 					else
+						ast_log(LOG_WARNING, "Unable to determine how many msgs are in the destination folder!\n");
+					break;
+				}
 				todircount = res;
 				ast_copy_string(tmp, fmt, sizeof(tmp));
 				stringp = tmp;
@@ -3547,7 +3553,6 @@ static int forward_message(struct ast_channel *chan, char *context, char *dir, i
 					res = ast_play_and_wait(chan, "vm-messages");
 				if (!res)
 					res = ast_play_and_wait(chan, "vm-saved"); */
-				if (!res)
 				res = ast_play_and_wait(chan, "vm-msgsaved");
 			}	
 		}