Applied patch from Matt Kraai which does the following:

 - adds case-insensitive matching in sed s/// epxressions
 - consolodates common regcomp code in grep & sed into bb_regcomp and put in
   utility.c
 - cleans up a bunch of cruft

1 files changed, 50 insertions, 83 deletions

diff --git a/editors/sed.c b/editors/sed.c
index 329f5ae..2fb243f 100644
--- a/editors/sed.c
+++ b/editors/sed.c
@@ -97,50 +97,6 @@ static const char sed_usage[] =
 #endif
 	;
 
-#if 0
-/* Nuke from here { */
-
-
-/* get_line_from_file() - This function reads an entire line from a text file
- *  * up to a newline. It returns a malloc'ed char * which must be stored and
- *   * free'ed  by the caller. */
-extern char *get_line_from_file(FILE *file)
-{
-	static const int GROWBY = 80; /* how large we will grow strings by */
-
-	int ch; 
-	int idx = 0;
-	char *linebuf = NULL;
-	int linebufsz = 0;
-
-	while (1) {
-		ch = fgetc(file);
-		if (ch == EOF)
-			break;
-		/* grow the line buffer as necessary */
-		if (idx > linebufsz-2)
-			linebuf = realloc(linebuf, linebufsz += GROWBY);
-		linebuf[idx++] = (char)ch;
-		if ((char)ch == '\n')
-			break;
-	}
-
-	if (idx == 0)
-		return NULL;
-
-	linebuf[idx] = 0;
-	return linebuf;
-}
-
-static void usage(const char *string)
-{
-	printf("usage: %s\n", string);
-	exit(0);
-}
-
-/* } to here when we integrate this into busybox */
-#endif
-
 static void destroy_cmd_strs()
 {
 	if (sed_cmds == NULL)
@@ -246,29 +202,15 @@ static int get_address(const char *str, int *line, regex_t **regex)
 		idx++;
 	}
 	else if (my_str[idx] == '/') {
-		int ret;
 		idx = index_of_next_unescaped_slash(idx, my_str);
-		if (idx == -1) {
-			free(my_str);
+		if (idx == -1)
 			exit_sed(1, "sed: unterminated match expression\n");
-		}
-		my_str[idx] = 0; /* shave off the trailing '/' */
-		my_str++; /* shave off the leading '/' */
-		*regex = (regex_t *)malloc(sizeof(regex_t));
-		if ((ret = regcomp(*regex, my_str, 0)) != 0) {
-			/* error handling if regular expression couldn't be compiled */
-			int errmsgsz = regerror(ret, *regex, NULL, 0);
-			char *errmsg = malloc(errmsgsz);
-			if (errmsg == NULL) {
-				exit_sed(1, "sed: memory error\n");
-			}
-			regerror(ret, *regex, errmsg, errmsgsz);
-			fprintf(stderr, "sed: %s\n", errmsg);
-			free(errmsg);
+		my_str[idx] = '\0';
+		*regex = (regex_t *)xmalloc(sizeof(regex_t));
+		if (bb_regcomp(*regex, my_str+1, REG_NEWLINE) != 0) {
+			free(my_str);
 			exit_sed(1, NULL);
 		}
-		my_str--; /* move my_str back so free() (below) won't barf */
-		idx++; /* advance idx one past the end of the /match/ */
 	}
 	else {
 		fprintf(stderr, "sed.c:get_address: no address found in string\n");
@@ -280,6 +222,15 @@ static int get_address(const char *str, int *line, regex_t **regex)
 	return idx;
 }
 
+static char *strdup_substr(const char *str, int start, int end)
+{
+	int size = end - start + 1;
+	char *newstr = xmalloc(size);
+	memcpy(newstr, str+start, size-1);
+	newstr[size-1] = '\0';
+	return newstr;
+}
+
 static void parse_cmd_str(struct sed_cmd *sed_cmd, const char *cmdstr)
 {
 	int idx = 0;
@@ -306,10 +257,11 @@ static void parse_cmd_str(struct sed_cmd *sed_cmd, const char *cmdstr)
 	sed_cmd->cmd = cmdstr[idx];
 	/* special-case handling for 's' */
 	if (sed_cmd->cmd == 's') {
-		int oldidx;
+		int oldidx, cflags = REG_NEWLINE;
+		char *match;
 		/* format for substitution is:
-		 *    s/match/replace/g
-		 *    |               |
+		 *    s/match/replace/gI
+		 *    |               ||
 		 *    mandatory       optional
 		 */
 
@@ -317,19 +269,41 @@ static void parse_cmd_str(struct sed_cmd *sed_cmd, const char *cmdstr)
 		if (cmdstr[++idx] != '/')
 			exit_sed(1, "sed: bad format in substitution expression\n");
 
-		/* get the substitution part */
-		idx += get_address(&cmdstr[idx], NULL, &sed_cmd->sub_match);
+		/* save the match string */
+		oldidx = idx+1;
+		idx = index_of_next_unescaped_slash(idx, cmdstr);
+		if (idx == -1)
+			exit_sed(1, "sed: bad format in substitution expression\n");
+		match = strdup_substr(cmdstr, oldidx, idx);
 
-		/* get the replacement part */
-		oldidx = idx;
+		/* save the replacement string */
+		oldidx = idx+1;
 		idx = index_of_next_unescaped_slash(idx, cmdstr);
-		sed_cmd->replace = (char *)malloc(idx - oldidx + 1);
-		strncpy(sed_cmd->replace, &cmdstr[oldidx], idx - oldidx);
-		sed_cmd->replace[idx - oldidx] = 0;
+		if (idx == -1)
+			exit_sed(1, "sed: bad format in substitution expression\n");
+		sed_cmd->replace = strdup_substr(cmdstr, oldidx, idx);
 
-		/* store the 'g' if present */
-		if (cmdstr[++idx] == 'g')
-			sed_cmd->sub_g = 1;
+		/* process the flags */
+		while (cmdstr[++idx]) {
+			switch (cmdstr[idx]) {
+			case 'g':
+				sed_cmd->sub_g = 1;
+				break;
+			case 'I':
+				cflags |= REG_ICASE;
+				break;
+			default:
+				exit_sed(1, "sed: bad option in substitution expression\n");
+			}
+		}
+			
+		/* compile the regex */
+		sed_cmd->sub_match = (regex_t *)xmalloc(sizeof(regex_t));
+		if (bb_regcomp(sed_cmd->sub_match, match, cflags) != 0) {
+			free(match);
+			exit_sed(1, NULL);
+		}
+		free(match);
 	}
 }
 
@@ -553,10 +527,3 @@ extern int sed_main(int argc, char **argv)
 	/* not reached */
 	return 0;
 }
-
-#ifdef TEST_SED
-int main(int argc, char **argv)
-{
-	return sed_main(argc, argv);
-}
-#endif