Merged revisions 124395 via svnmerge from
authorTilghman Lesher <tilghman@meg.abyt.es>
Fri, 20 Jun 2008 22:04:37 +0000 (22:04 +0000)
committerTilghman Lesher <tilghman@meg.abyt.es>
Fri, 20 Jun 2008 22:04:37 +0000 (22:04 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r124395 | tilghman | 2008-06-20 17:02:55 -0500 (Fri, 20 Jun 2008) | 3 lines

If the last character in a string to be parsed is the delimiter, then we should
count that final empty string as an additional argument.

........

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

main/app.c

index ebdd237..852e3b8 100644 (file)
@@ -1026,7 +1026,7 @@ int ast_app_group_list_unlock(void)
 unsigned int ast_app_separate_args(char *buf, char delim, char **array, int arraylen)
 {
        int argc;
-       char *scan;
+       char *scan, *wasdelim = NULL;
        int paren = 0, quote = 0;
 
        if (!buf || !array || !arraylen)
@@ -1053,14 +1053,18 @@ unsigned int ast_app_separate_args(char *buf, char delim, char **array, int arra
                                /* Literal character, don't parse */
                                memmove(scan, scan + 1, strlen(scan));
                        } else if ((*scan == delim) && !paren && !quote) {
+                               wasdelim = scan;
                                *scan++ = '\0';
                                break;
                        }
                }
        }
 
-       if (*scan)
+       /* If the last character in the original string was the delimiter, then
+        * there is one additional argument. */
+       if (*scan || (scan > buf && (scan - 1) == wasdelim)) {
                array[argc++] = scan;
+       }
 
        return argc;
 }