[asterisk-commits] kpfleming: branch 1.4 r47641 - /branches/1.4/main/term.c

[asterisk-commits at lists.digium.com]

Author: kpfleming
Date: Tue Nov 14 18:19:05 2006
New Revision: 47641

URL: http://svn.digium.com/view/asterisk?view=rev&rev=47641
Log:
more formatting cleanup, and avoid running off the end of the string

Modified:
    branches/1.4/main/term.c

Modified: branches/1.4/main/term.c
URL: http://svn.digium.com/view/asterisk/branches/1.4/main/term.c?view=diff&rev=47641&r1=47640&r2=47641
==============================================================================
--- branches/1.4/main/term.c (original)
+++ branches/1.4/main/term.c Tue Nov 14 18:19:05 2006
@@ -266,28 +266,26 @@
 
 /* filter escape sequences */
 void term_filter_escapes(char *line)
- {
-	 int i;
-   
-	 for (i = 0; i < strlen(line); i++) {
+{
+	int i;
+	int len = strlen(line);
+
+	for (i = 0; i < len; i++) {
 		if (line[i] != ESC)
 			continue;
-		if (line[i + 1] == '\x5b') {
+		if ((i < (len - 2)) &&
+		    (line[i + 1] == 0x5B)) {
 			switch (line[i + 2]) {
-		 	case '\x30':
-			case '\x31':
-			case '\x33':
-				break;
-			default:
-				/* replace ESC with a space */
-				line[i] = ' ';
+		 	case 0x30:
+			case 0x31:
+			case 0x33:
+				continue;
 			}
-		} else {
-			/* replace ESC with a space */
-			line[i] = ' ';
-		}
-	 }
- }
+		}
+		/* replace ESC with a space */
+		line[i] = ' ';
+	}
+}
 
 char *term_prep(void)
 {