Properly handle | within ()'s when doing app arg parsing... (bug #4911)

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

app.c

@@ -1066,9 +1066,9 @@ int ast_app_group_match_get_count(char *groupmatch, char *category)
 
 int ast_separate_app_args(char *buf, char delim, char **array, int arraylen)
 {
-	int x;
+	int argc;
 	char *scan;
-	char delims[2];
+	int paren = 0;
 
 	if (!buf || !array || !arraylen)
 		return 0;
@@ -1076,21 +1076,26 @@ int ast_separate_app_args(char *buf, char delim, char **array, int arraylen)
 	memset(array, 0, arraylen * sizeof(*array));
 
 	scan = buf;
-	delims[0] = delim;
+
-	delims[1] = '\0';
+	for (argc = 0; *scan && (argc < arraylen - 1); argc++) {
-	x = 0;
+		array[argc] = scan;
-
+		for (; *scan; scan++) {
-	while (x < arraylen - 1) {
+			if (*scan == '(')
-		array[x] = strsep(&scan, delims);
+				paren++;
-		x++;
+			else if (*scan == ')') {
-		if (!scan)
+				if (paren)
-			break;
+					paren--;
+			} else if ((*scan == delim) && !paren) {
+				*scan++ = '\0';
+				break;
+			}
+		}
 	}
 
-	if (scan)
+	if (*scan)
-		array[x++] = scan;
+		array[argc++] = scan;
 
-	return x;
+	return argc;
 }
 
 enum AST_LOCK_RESULT ast_lock_path(const char *path)