[asterisk-commits] tilghman: branch 1.6.1 r219988 - in /branches/1.6.1: ./ apps/app_directory.c

[SVN commits to the Asterisk project]

Author: tilghman
Date: Thu Sep 24 02:44:46 2009
New Revision: 219988

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=219988
Log:
Merged revisions 219987 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

........
  r219987 | tilghman | 2009-09-24 02:39:44 -0500 (Thu, 24 Sep 2009) | 8 lines
  
  Fix two possible crashes, one only in 1.6.1 and one in 1.6.1 forward.
  (closes issue #15739)
   Reported by: DLNoah, jeffg
   Patches: 
         20090914__issue15739.diff.txt uploaded by tilghman (license 14)
         20090922__issue15739.diff.txt uploaded by tilghman (license 14)
   Tested by: DLNoah, jeffg
........

Modified:
    branches/1.6.1/   (props changed)
    branches/1.6.1/apps/app_directory.c

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

Modified: branches/1.6.1/apps/app_directory.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.6.1/apps/app_directory.c?view=diff&rev=219988&r1=219987&r2=219988
==============================================================================
--- branches/1.6.1/apps/app_directory.c (original)
+++ branches/1.6.1/apps/app_directory.c Thu Sep 24 02:44:46 2009
@@ -473,6 +473,11 @@
 		strsep(&bufptr, ",");
 		pos = strsep(&bufptr, ",");
 
+		/* No name to compare against */
+		if (ast_strlen_zero(pos)) {
+			continue;
+		}
+
 		res = 0;
 		if (ast_test_flag(&flags, OPT_LISTBYLASTNAME)) {
 			res = check_match(&item, pos, v->name, ext, 0 /* use_first_name */);
@@ -645,7 +650,7 @@
 	int res = 0, digit = 3;
 	struct ast_config *cfg, *ucfg;
 	const char *dirintro;
-	char *parse, *opts[OPT_ARG_ARRAY_SIZE];
+	char *parse, *opts[OPT_ARG_ARRAY_SIZE] = { "", };
 	struct ast_flags flags = { 0 };
 	struct ast_flags config_flags = { 0 };
 	enum { FIRST, LAST, BOTH } which = LAST;