[asterisk-commits] tilghman: trunk r77851 - /trunk/res/res_agi.c

[SVN commits to the Asterisk project]

Author: tilghman
Date: Tue Jul 31 23:36:15 2007
New Revision: 77851

URL: http://svn.digium.com/view/asterisk?view=rev&rev=77851
Log:
Twould help if we actually defined ->mod before comparing against it (reported and fixed by Juggie via IRC).

Modified:
    trunk/res/res_agi.c

Modified: trunk/res/res_agi.c
URL: http://svn.digium.com/view/asterisk/trunk/res/res_agi.c?view=diff&rev=77851&r1=77850&r2=77851
==============================================================================
--- trunk/res/res_agi.c (original)
+++ trunk/res/res_agi.c Tue Jul 31 23:36:15 2007
@@ -1662,6 +1662,7 @@
 	ast_join(fullcmd, sizeof(fullcmd), cmd->cmda);
 
  	if (!find_command(cmd->cmda,1)) {
+		cmd->mod = mod;
 		AST_RWLIST_WRLOCK(&agi_commands);
 		AST_LIST_INSERT_TAIL(&agi_commands, cmd, list);
 		AST_RWLIST_UNLOCK(&agi_commands);
@@ -1728,26 +1729,26 @@
 	AST_RWLIST_TRAVERSE(&agi_commands, e, list) {
 		if (!e->cmda[0])
 			break;
-			/* start optimistic */
-			match = 1;
-			for (y = 0; match && cmds[y]; y++) {
-				/* If there are no more words in the command (and we're looking for
-				   an exact match) or there is a difference between the two words,
-				   then this is not a match */
-				if (!e->cmda[y] && !exact)
-					break;
-				/* don't segfault if the next part of a command doesn't exist */
-				if (!e->cmda[y])
-					return NULL;
-				if (strcasecmp(e->cmda[y], cmds[y]))
-					match = 0;
-			}
-			/* If more words are needed to complete the command then this is not
-			   a candidate (unless we're looking for a really inexact answer  */
-			if ((exact > -1) && e->cmda[y])
+		/* start optimistic */
+		match = 1;
+		for (y = 0; match && cmds[y]; y++) {
+			/* If there are no more words in the command (and we're looking for
+			   an exact match) or there is a difference between the two words,
+			   then this is not a match */
+			if (!e->cmda[y] && !exact)
+				break;
+			/* don't segfault if the next part of a command doesn't exist */
+			if (!e->cmda[y])
+				return NULL;
+			if (strcasecmp(e->cmda[y], cmds[y]))
 				match = 0;
-			if (match)
-				return e;
+		}
+		/* If more words are needed to complete the command then this is not
+		   a candidate (unless we're looking for a really inexact answer  */
+		if ((exact > -1) && e->cmda[y])
+			match = 0;
+		if (match)
+			return e;
 	}
 	AST_RWLIST_UNLOCK(&agi_commands);
 	return NULL;