[asterisk-commits] tilghman: branch 1.6.0 r172742 - in /branches/1.6.0: ./ apps/app_voicemail.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Sat Jan 31 20:45:05 CST 2009


Author: tilghman
Date: Sat Jan 31 20:45:05 2009
New Revision: 172742

URL: http://svn.digium.com/svn-view/asterisk?view=rev&rev=172742
Log:
Merged revisions 172741 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

........
  r172741 | tilghman | 2009-01-31 20:44:23 -0600 (Sat, 31 Jan 2009) | 4 lines
  
  Blank argument crashes Asterisk
  (closes issue #14377)
   Reported by: amorsen
........

Modified:
    branches/1.6.0/   (props changed)
    branches/1.6.0/apps/app_voicemail.c

Propchange: branches/1.6.0/
------------------------------------------------------------------------------
Binary property 'trunk-merged' - no diff available.

Modified: branches/1.6.0/apps/app_voicemail.c
URL: http://svn.digium.com/svn-view/asterisk/branches/1.6.0/apps/app_voicemail.c?view=diff&rev=172742&r1=172741&r2=172742
==============================================================================
--- branches/1.6.0/apps/app_voicemail.c (original)
+++ branches/1.6.0/apps/app_voicemail.c Sat Jan 31 20:45:05 2009
@@ -8366,6 +8366,11 @@
 	);
 
 	AST_NONSTANDARD_APP_ARGS(arg, args, '@');
+
+	if (ast_strlen_zero(arg.mbox)) {
+		ast_log(LOG_ERROR, "MAILBOX_EXISTS requires an argument (<mailbox>[@<context>])\n");
+		return -1;
+	}
 
 	ast_copy_string(buf, find_user(&svm, ast_strlen_zero(arg.context) ? "default" : arg.context, arg.mbox) ? "1" : "0", len);
 	return 0;




More information about the asterisk-commits mailing list