From 9e4c207dd30489be44ee77c41f0f97d0290b53c5 Mon Sep 17 00:00:00 2001
From: "Kevin P. Fleming" <kpfleming@digium.com>
Date: Fri, 11 Nov 2005 01:01:19 +0000
Subject: [PATCH] ugh... another broken commit

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@7066 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---
 ChangeLog | 2 ++
 app.c     | 3 ++-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 1e76812a1e..bd2c879146 100755
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,7 @@
 2005-11-10  Kevin P. Fleming  <kpfleming@digium.com>
 
+	* app.c (ast_app_parse_options): ok, so we aren't all perfect... let's make the while loop actually work properly here (issue #5684)
+
 	* apps/app_disa.c (disa_exec): correct password file parsing (issue #5676)
 
 	* apps/app_meetme.c (conf_run): don't restrict admin users from joining a locked conference (issue #5680)
diff --git a/app.c b/app.c
index bcb5c22598..ea83601c2a 100755
--- a/app.c
+++ b/app.c
@@ -1544,7 +1544,8 @@ int ast_app_parse_options(const struct ast_app_option *options, struct ast_flags
 		if (*s == '(') {
 			/* Has argument */
 			arg = ++s;
-			while (*s && (*s++ != ')'));
+			while (*s && (*s != ')'))
+				s++;
 			if (*s) {
 				if (argloc)
 					args[argloc - 1] = arg;