Be more flexible with whitespace on AMI action headers.

Previously, this code required exactly one space to be after the ':' in headers
for an AMI action.  This now makes whitespace optional, and allows whitespace that
is there to vary in amount.

(closes issue #17862)
Reported by: cmoye
Patches:
      manager.c.patch_trunk uploaded by cmoye (license 858)
      manager.c.patch_1.8 uploaded by cmoye (license 858)
Tested by: cmoye

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

diff --git a/main/manager.c b/main/manager.c
index e9007d80441b998b5c4e1c7544c316480cb9d678..8aa355def140d2d2d3202ff4199aebcd687247dd 100644 (file)
--- a/main/manager.c
+++ b/main/manager.c
@@ -1697,8 +1697,9 @@ static const char *__astman_get_header(const struct message *m, char *var, int m
 
        for (x = 0; x < m->hdrcount; x++) {
                const char *h = m->headers[x];
-               if (!strncasecmp(var, h, l) && h[l] == ':' && h[l+1] == ' ') {
-                       const char *value = h + l + 2;
+               if (!strncasecmp(var, h, l) && h[l] == ':') {
+                       const char *value = h + l + 1;
+                       value = ast_skip_blanks(value); /* ignore leading spaces in the value */
                        /* found a potential candidate */
                        if (mode & GET_HEADER_SKIP_EMPTY && ast_strlen_zero(value))
                                continue;       /* not interesting */