[asterisk-commits] murf: branch 1.4 r57426 - in /branches/1.4: pbx/pbx_ael.c utils/ael_main.c

asterisk-commits at lists.digium.com asterisk-commits at lists.digium.com
Thu Mar 1 22:21:36 MST 2007


Author: murf
Date: Thu Mar  1 23:21:36 2007
New Revision: 57426

URL: http://svn.digium.com/view/asterisk?view=rev&rev=57426
Log:
I almost had comma escapes right, but 9184 points out the problem-- the escape is removed by pbx_config, and pbx_ael should also, before sending it down into the pbx engine. Also, you have to insert it back in, if you are generating extensions.conf code from the AEL.

Modified:
    branches/1.4/pbx/pbx_ael.c
    branches/1.4/utils/ael_main.c

Modified: branches/1.4/pbx/pbx_ael.c
URL: http://svn.digium.com/view/asterisk/branches/1.4/pbx/pbx_ael.c?view=diff&rev=57426&r1=57425&r2=57426
==============================================================================
--- branches/1.4/pbx/pbx_ael.c (original)
+++ branches/1.4/pbx/pbx_ael.c Thu Mar  1 23:21:36 2007
@@ -151,11 +151,19 @@
 static void substitute_commas(char *str)
 {
 	char *p = str;
+	
 	while (p && *p)
 	{
 		if (*p == ',' && ((p != str && *(p-1) != '\\')
 				|| p == str))
 			*p = '|';
+		if (*p == '\\' && *(p+1) == ',') { /* learning experience: the '\,' is turned into just ',' by pbx_config; So we need to do the same */
+			char *q = p;
+			while (*q) {  /* move the ',' and everything after it up 1 char */
+				*q = *(q+1);
+				q++;
+			}
+		}
 		p++;
 	}
 }

Modified: branches/1.4/utils/ael_main.c
URL: http://svn.digium.com/view/asterisk/branches/1.4/utils/ael_main.c?view=diff&rev=57426&r1=57425&r2=57426
==============================================================================
--- branches/1.4/utils/ael_main.c (original)
+++ branches/1.4/utils/ael_main.c Thu Mar  1 23:21:36 2007
@@ -181,6 +181,8 @@
 
 	if( dump_extensions && dumpfile ) {
 		struct namelist *n;
+		char *data2,*data3=0;
+		int commacount = 0;
 
 		if( FIRST_TIME ) {
 			FIRST_TIME = 0;
@@ -215,10 +217,42 @@
 			filter_newlines((char*)data);
 			filter_leading_space_from_exprs((char*)data);
 
+			/* compiling turns commas into vertical bars in the app data, and also removes the backslash from before escaped commas;
+			   we have to restore the escaping backslash in front of any commas; the vertical bars are OK to leave as-is */
+			for (data2 = data; *data2; data2++) {
+				if (*data2 == ',')
+					commacount++;  /* we need to know how much bigger the string will grow-- one backslash for each comma  */
+			}
+			if (commacount) 
+			{
+				char *d3,*d4;
+				
+				data2 = (char*)malloc(strlen(data)+commacount+1);
+				data3 = data;
+				d3 = data;
+				d4 = data2;
+				while (*d3) {
+					if (*d3 == ',') {
+						*d4++ = '\\'; /* put a backslash in front of each comma */
+						*d4++ = *d3++;
+					} else
+						*d4++ = *d3++;  /* or just copy the char */
+				}
+				*d4++ = 0;  /* cap off the new string */
+				data = data2;
+			} else
+				data2 = 0;
+			
 			if( strcmp(label,"(null)") != 0  )
 				fprintf(dumpfile,"exten => %s,%d(%s),%s(%s)\n", extension, priority, label, application, (char*)data);
 			else
 				fprintf(dumpfile,"exten => %s,%d,%s(%s)\n", extension, priority, application, (char*)data);
+
+			if (data2) {
+				free(data2);
+				data2 = 0;
+				data = data3; /* restore data to pre-messedup state */
+			}
 
 		} else {
 



More information about the asterisk-commits mailing list