[asterisk-commits] russell: trunk r284066 - in /trunk: ./ main/manager.c

[SVN commits to the Asterisk project]

Author: russell
Date: Sat Aug 28 16:30:25 2010
New Revision: 284066

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=284066
Log:
Merged revisions 284065 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r284065 | russell | 2010-08-28 16:29:45 -0500 (Sat, 28 Aug 2010) | 13 lines
  
  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
........

Modified:
    trunk/   (props changed)
    trunk/main/manager.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: trunk/main/manager.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/manager.c?view=diff&rev=284066&r1=284065&r2=284066
==============================================================================
--- trunk/main/manager.c (original)
+++ trunk/main/manager.c Sat Aug 28 16:30:25 2010
@@ -1697,8 +1697,9 @@
 
 	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 */