Fix calls to {m,c,re}alloc so that they use x{m,c,re}alloc instead of
[oweals/busybox.git] / sed.c
diff --git a/sed.c b/sed.c
index 8ae34f8c7b7c49ee2b530c5247a66bb447d2ad83..0e0d7f58c46e5f4d430828d3562bfe36c7e4f238 100644 (file)
--- a/sed.c
+++ b/sed.c
        Supported features and commands in this version of sed:
 
         - comments ('#')
-        - Address matching: num|/matchstr/[,num|/matchstr/|$]command
-        - Commands: p, d, s/match/replace/[g]
+        - address matching: num|/matchstr/[,num|/matchstr/|$]command
+        - commands: (p)rint, (d)elete, (s)ubstitue (with g & I flags)
+        - edit commands: (a)ppend, (i)nsert, (c)hange
+        - backreferences in substitution expressions (\1, \2...\9)
         
         (Note: Specifying an address (range) to match is *optional*; commands
         default to the whole pattern space if no specific address match was
@@ -42,7 +44,6 @@
 */
 
 #include <stdio.h>
-#include <stdlib.h> /* for realloc() */
 #include <unistd.h> /* for getopt() */
 #include <regex.h>
 #include <string.h> /* for strdup() */
@@ -50,6 +51,9 @@
 #include <ctype.h> /* for isspace() */
 #include "internal.h"
 
+#define bb_need_full_version
+#define BB_DECLARE_EXTERN
+#include "messages.c"
 
 /* externs */
 extern int optind; /* in unistd.h */
@@ -58,6 +62,11 @@ extern char *optarg; /* ditto */
 /* options */
 static int be_quiet = 0;
 
+static const int SUB_G = 1 << 0;
+#ifdef BB_FEATURE_SED_PATTERN_SPACE
+static const int SUB_P = 1 << 1;
+#endif
+
 struct sed_cmd {
 
        /* address storage */
@@ -72,7 +81,17 @@ struct sed_cmd {
        /* substitution command specific fields */
        regex_t *sub_match; /* sed -e 's/sub_match/replace/' */
        char *replace; /* sed -e 's/sub_match/replace/' XXX: who will hold the \1 \2 \3s? */
-       unsigned int sub_g:1; /* sed -e 's/foo/bar/g' (global) */
+       unsigned int num_backrefs:4; /* how many back references (\1..\9) */
+                       /* Note:  GNU/POSIX sed does not save more than nine backrefs, so
+                        * we only use 4 bits to hold the number */
+#ifndef BB_FEATURE_SED_PATTERN_SPACE
+       unsigned int sub_flags:1; /* sed -e 's/foo/bar/g' (global) */
+#else
+       unsigned int sub_flags:2; /* sed -e 's/foo/bar/gp' (global/pattern) */
+#endif
+
+       /* edit command (a,i,c) speicific field */
+       char *editline;
 };
 
 /* globals */
@@ -81,23 +100,7 @@ static int ncmds = 0; /* number of sed commands */
 
 /*static char *cur_file = NULL;*/ /* file currently being processed XXX: do I need this? */
 
-static const char sed_usage[] =
-       "sed [-Vhnef] pattern [files...]\n"
-#ifndef BB_FEATURE_TRIVIAL_HELP
-       "\n"
-       "-n\tsuppress automatic printing of pattern space\n"
-       "-e script\tadd the script to the commands to be executed\n"
-       "-f scriptfile\tadd the contents of script-file to the commands to be executed\n"
-       "-h\tdisplay this help message\n"
-       "-V\toutput version information and exit\n"
-       "\n"
-       "If no -e or -f is given, the first non-option argument is taken as the\n"
-       "sed script to interpret. All remaining arguments are names of input\n"
-       "files; if no input files are specified, then the standard input is read.\n"
-#endif
-       ;
-
-#if 0
+#ifdef BB_FEATURE_CLEAN_UP
 static void destroy_cmd_strs()
 {
        if (sed_cmds == NULL)
@@ -128,11 +131,13 @@ static void destroy_cmd_strs()
 }
 #endif
 
+#if 0
 /*
  * trim_str - trims leading and trailing space from a string
  * 
  * Note: This returns a malloc'ed string so you must store and free it
  * XXX: This should be in the utility.c file.
+ * XXX: This is now obsolete. Maybe it belongs nowhere.
  */
 static char *trim_str(const char *str)
 {
@@ -152,26 +157,43 @@ static char *trim_str(const char *str)
         *
         * you know, a strrspn() would really be nice cuz then we could say:
         * 
-        * retstr[strlen(retstr) - strrspn(retstr, " \n\t\v") + 1] = 0;
+        * retstr[strrspn(retstr, " \n\t\v") + 1] = 0;
         */
        
        return retstr;
 }
+#endif
 
+#if 0
 /*
- * index_of_unescaped_slash - walks left to right through a string beginning
- * at a specified index and returns the index of the next unescaped slash.
+ * strrspn - works just like strspn() but goes from right to left instead of
+ * left to right
+ */
+static size_t strrspn(const char *s, const char *accept)
+{
+       size_t i = strlen(s);
+
+       while (strchr(accept, s[--i]))
+               ;
+
+       return i;
+}
+#endif
+
+/*
+ * index_of_next_unescaped_slash - walks left to right through a string
+ * beginning at a specified index and returns the index of the next forward
+ * slash ('/') not preceeded by a backslash ('\').
  */
 static int index_of_next_unescaped_slash(const char *str, int idx)
 {
-       do {
-               idx++;
-               /* test if we've hit the end */
-               if (str[idx] == 0)
-                       return -1;
-       } while (str[idx] != '/' && str[idx - 1] != '\\');
+       for ( ; str[idx]; idx++) {
+               if (str[idx] == '/' && str[idx-1] != '\\')
+                       return idx;
+       }
 
-       return idx;
+       /* if we make it to here, we've hit the end of the string */
+       return -1;
 }
 
 /*
@@ -188,24 +210,23 @@ static int get_address(const char *str, int *line, regex_t **regex)
                } while (isdigit(my_str[idx]));
                my_str[idx] = 0;
                *line = atoi(my_str);
-               *regex = NULL;
        }
        else if (my_str[idx] == '$') {
                *line = -1;
-               *regex = NULL;
                idx++;
        }
        else if (my_str[idx] == '/') {
-               idx = index_of_next_unescaped_slash(my_str, idx);
+               idx = index_of_next_unescaped_slash(my_str, ++idx);
                if (idx == -1)
                        fatalError("unterminated match expression\n");
                my_str[idx] = '\0';
                *regex = (regex_t *)xmalloc(sizeof(regex_t));
                xregcomp(*regex, my_str+1, REG_NEWLINE);
+               idx++; /* so it points to the next character after the last '/' */
        }
        else {
-               fprintf(stderr, "sed.c:get_address: no address found in string\n");
-               fprintf(stderr, "\t(you probably didn't check the string you passed me)\n");
+               errorMsg("get_address: no address found in string\n"
+                               "\t(you probably didn't check the string you passed me)\n");
                idx = -1;
        }
 
@@ -222,16 +243,17 @@ static char *strdup_substr(const char *str, int start, int end)
        return newstr;
 }
 
-static void parse_subst_cmd(struct sed_cmd *sed_cmd, const char *substr)
+static int parse_subst_cmd(struct sed_cmd *sed_cmd, const char *substr)
 {
        int oldidx, cflags = REG_NEWLINE;
        char *match;
        int idx = 0;
+       int j;
 
        /*
         * the string that gets passed to this function should look like this:
-        *    s/match/replace/gI
-        *    ||     |        ||
+        *    s/match/replace/gIp
+        *    ||     |        |||
         *    mandatory       optional
         *
         *    (all three of the '/' slashes are mandatory)
@@ -243,14 +265,26 @@ static void parse_subst_cmd(struct sed_cmd *sed_cmd, const char *substr)
 
        /* save the match string */
        oldidx = idx+1;
-       idx = index_of_next_unescaped_slash(substr, idx);
+       idx = index_of_next_unescaped_slash(substr, ++idx);
        if (idx == -1)
                fatalError("bad format in substitution expression\n");
        match = strdup_substr(substr, oldidx, idx);
 
+       /* determine the number of back references in the match string */
+       /* Note: we compute this here rather than in the do_subst_command()
+        * function to save processor time, at the expense of a little more memory
+        * (4 bits) per sed_cmd */
+       
+       /* sed_cmd->num_backrefs = 0; */ /* XXX: not needed? --apparently not */ 
+       for (j = 0; match[j]; j++) {
+               /* GNU/POSIX sed does not save more than nine backrefs */
+               if (match[j] == '\\' && match[j+1] == '(' && sed_cmd->num_backrefs < 9)
+                       sed_cmd->num_backrefs++;
+       }
+
        /* save the replacement string */
        oldidx = idx+1;
-       idx = index_of_next_unescaped_slash(substr, idx);
+       idx = index_of_next_unescaped_slash(substr, ++idx);
        if (idx == -1)
                fatalError("bad format in substitution expression\n");
        sed_cmd->replace = strdup_substr(substr, oldidx, idx);
@@ -258,24 +292,108 @@ static void parse_subst_cmd(struct sed_cmd *sed_cmd, const char *substr)
        /* process the flags */
        while (substr[++idx]) {
                switch (substr[idx]) {
-               case 'g':
-                       sed_cmd->sub_g = 1;
-                       break;
-               case 'I':
-                       cflags |= REG_ICASE;
-                       break;
-               default:
-                       fatalError("bad option in substitution expression\n");
+                       case 'g':
+                               sed_cmd->sub_flags = SUB_G;
+                               break;
+                       case 'I':
+                               cflags |= REG_ICASE;
+                               break;
+#ifdef BB_FEATURE_SED_PATTERN_SPACE
+                       case 'p':
+                               sed_cmd->sub_flags = SUB_P;
+                               break;
+#endif
+                       default:
+                               /* any whitespace or semicolon trailing after a s/// is ok */
+                               if (strchr("; \t\v\n\r", substr[idx]))
+                                       goto out;
+                               /* else */
+                               fatalError("bad option in substitution expression\n");
                }
        }
-               
-       /* compile the regex */
+
+out:   
+       /* compile the match string into a regex */
        sed_cmd->sub_match = (regex_t *)xmalloc(sizeof(regex_t));
        xregcomp(sed_cmd->sub_match, match, cflags);
        free(match);
+
+       return idx;
 }
 
-static void parse_cmd_str(struct sed_cmd *sed_cmd, const char *cmdstr)
+static int parse_edit_cmd(struct sed_cmd *sed_cmd, const char *editstr)
+{
+       int idx = 0;
+       int slashes_eaten = 0;
+       char *ptr; /* shorthand */
+
+       /*
+        * the string that gets passed to this function should look like this:
+        *
+        *    need one of these 
+        *    |
+        *    |    this backslash (immediately following the edit command) is mandatory
+        *    |    |
+        *    [aic]\
+        *    TEXT1\
+        *    TEXT2\
+        *    TEXTN
+        *
+        * as soon as we hit a TEXT line that has no trailing '\', we're done.
+        * this means a command like:
+        *
+        * i\
+        * INSERTME
+        *
+        * is a-ok.
+        *
+        */
+
+       if (editstr[1] != '\\' && (editstr[2] != '\n' || editstr[2] != '\r'))
+               fatalError("bad format in edit expression\n");
+
+       /* store the edit line text */
+       /* make editline big enough to accomodate the extra '\n' we will tack on
+        * to the end */
+       sed_cmd->editline = xmalloc(strlen(&editstr[3]) + 2);
+       strcpy(sed_cmd->editline, &editstr[3]);
+       ptr = sed_cmd->editline;
+
+       /* now we need to go through * and: s/\\[\r\n]$/\n/g on the edit line */
+       while (ptr[idx]) {
+               while (ptr[idx] != '\\' && (ptr[idx+1] != '\n' || ptr[idx+1] != '\r')) {
+                       idx++;
+                       if (!ptr[idx]) {
+                               goto out;
+                       }
+               }
+               /* move the newline over the '\' before it (effectively eats the '\') */
+               memmove(&ptr[idx], &ptr[idx+1], strlen(&ptr[idx+1]));
+               ptr[strlen(ptr)-1] = 0;
+               slashes_eaten++;
+               /* substitue \r for \n if needed */
+               if (ptr[idx] == '\r')
+                       ptr[idx] = '\n';
+       }
+
+out:
+       ptr[idx] = '\n';
+       ptr[idx+1] = 0;
+
+       /* this accounts for discrepancies between the modified string and the
+        * original string passed in to this function */
+       idx += slashes_eaten;
+
+       /* this accounts for the fact that A) we started at index 3, not at index
+        * 0  and B) that we added an extra '\n' at the end (if you think the next
+        * line should read 'idx += 4' remember, arrays are zero-based) */
+
+       idx += 3;
+
+       return idx;
+}
+
+static char *parse_cmd_str(struct sed_cmd *sed_cmd, const char *cmdstr)
 {
        int idx = 0;
 
@@ -285,6 +403,7 @@ static void parse_cmd_str(struct sed_cmd *sed_cmd, const char *cmdstr)
         *            part1 part2  part3
         */
 
+
        /* first part (if present) is an address: either a number or a /regex/ */
        if (isdigit(cmdstr[idx]) || cmdstr[idx] == '/')
                idx = get_address(cmdstr, &sed_cmd->beg_line, &sed_cmd->beg_match);
@@ -296,31 +415,54 @@ static void parse_cmd_str(struct sed_cmd *sed_cmd, const char *cmdstr)
        /* last part (mandatory) will be a command */
        if (cmdstr[idx] == '\0')
                fatalError("missing command\n");
-       if (!strchr("pds", cmdstr[idx])) /* <-- XXX add new commands here */
+       if (!strchr("pdsaic", cmdstr[idx])) /* <-- XXX add new commands here */
                fatalError("invalid command\n");
        sed_cmd->cmd = cmdstr[idx];
 
        /* special-case handling for (s)ubstitution */
-       if (sed_cmd->cmd == 's')
-               parse_subst_cmd(sed_cmd, &cmdstr[idx]);
+       if (sed_cmd->cmd == 's') {
+               idx += parse_subst_cmd(sed_cmd, &cmdstr[idx]);
+       }
+       /* special-case handling for (a)ppend, (i)nsert, and (c)hange */
+       else if (strchr("aic", cmdstr[idx])) {
+               if (sed_cmd->end_line || sed_cmd->end_match)
+                       fatalError("only a beginning address can be specified for edit commands\n");
+               idx += parse_edit_cmd(sed_cmd, &cmdstr[idx]);
+       }
+       /* if it was a single-letter command (such as 'p' or 'd') we need to
+        * increment the index past that command */
+       else
+               idx++;
+
+       /* give back whatever's left over */
+       return (char *)&cmdstr[idx];
 }
 
 static void add_cmd_str(const char *cmdstr)
 {
-       char *my_cmdstr = trim_str(cmdstr);
+       char *mystr = (char *)cmdstr;
 
-       /* if this is a comment, don't even bother */
-       if (my_cmdstr[0] == '#') {
-               free(my_cmdstr);
-               return;
-       }
+       do {
 
-       /* grow the array */
-       sed_cmds = realloc(sed_cmds, sizeof(struct sed_cmd) * (++ncmds));
-       /* zero new element */
-       memset(&sed_cmds[ncmds-1], 0, sizeof(struct sed_cmd));
-       /* load command string into new array element */
-       parse_cmd_str(&sed_cmds[ncmds-1], my_cmdstr);
+               /* trim leading whitespace and semicolons */
+               memmove(mystr, &mystr[strspn(mystr, "; \n\r\t\v")], strlen(mystr));
+               /* if we ate the whole thing, that means there was just trailing
+                * whitespace or a final / no-op semicolon. either way, get out */
+               if (strlen(mystr) == 0)
+                       return;
+               /* if this is a comment, jump past it and keep going */
+               if (mystr[0] == '#') {
+                       mystr = strpbrk(mystr, ";\n\r");
+                       continue;
+               }
+               /* grow the array */
+               sed_cmds = xrealloc(sed_cmds, sizeof(struct sed_cmd) * (++ncmds));
+               /* zero new element */
+               memset(&sed_cmds[ncmds-1], 0, sizeof(struct sed_cmd));
+               /* load command string into new array element, get remainder */
+               mystr = parse_cmd_str(&sed_cmds[ncmds-1], mystr);
+
+       } while (mystr && strlen(mystr));
 }
 
 
@@ -328,18 +470,127 @@ static void load_cmd_file(char *filename)
 {
        FILE *cmdfile;
        char *line;
+       char *nextline;
 
        cmdfile = fopen(filename, "r");
        if (cmdfile == NULL)
                fatalError(strerror(errno));
 
        while ((line = get_line_from_file(cmdfile)) != NULL) {
-               line[strlen(line)-1] = 0; /* eat newline */
+               /* if a line ends with '\' it needs the next line appended to it */
+               while (line[strlen(line)-2] == '\\' &&
+                               (nextline = get_line_from_file(cmdfile)) != NULL) {
+                       line = xrealloc(line, strlen(line) + strlen(nextline) + 1);
+                       strcat(line, nextline);
+                       free(nextline);
+               }
+               /* eat trailing newline (if any) --if I don't do this, edit commands
+                * (aic) will print an extra newline */
+               if (line[strlen(line)-1] == '\n')
+                       line[strlen(line)-1] = 0;
                add_cmd_str(line);
                free(line);
        }
 }
 
+static void print_subst_w_backrefs(const char *line, const char *replace, regmatch_t *regmatch)
+{
+       int i;
+
+       /* go through the replacement string */
+       for (i = 0; replace[i]; i++) {
+               /* if we find a backreference (\1, \2, etc.) print the backref'ed * text */
+               if (replace[i] == '\\' && isdigit(replace[i+1])) {
+                       int j;
+                       char tmpstr[2];
+                       int backref;
+                       ++i; /* i now indexes the backref number, instead of the leading slash */
+                       tmpstr[0] = replace[i];
+                       tmpstr[1] = 0;
+                       backref = atoi(tmpstr);
+                       /* print out the text held in regmatch[backref] */
+                       for (j = regmatch[backref].rm_so; j < regmatch[backref].rm_eo; j++)
+                               fputc(line[j], stdout);
+               }
+
+               /* if we find a backslash escaped character, print the character */
+               else if (replace[i] == '\\') {
+                       ++i;
+                       fputc(replace[i], stdout);
+               }
+
+               /* if we find an unescaped '&' print out the whole matched text.
+                * fortunately, regmatch[0] contains the indicies to the whole matched
+                * expression (kinda seems like it was designed for just such a
+                * purpose...) */
+               else if (replace[i] == '&' && replace[i-1] != '\\') {
+                       int j;
+                       for (j = regmatch[0].rm_so; j < regmatch[0].rm_eo; j++)
+                               fputc(line[j], stdout);
+               }
+               /* nothing special, just print this char of the replacement string to stdout */
+               else
+                       fputc(replace[i], stdout);
+       }
+}
+
+static int do_subst_command(const struct sed_cmd *sed_cmd, const char *line)
+{
+       int altered = 0;
+
+       /* we only substitute if the substitution 'search' expression matches */
+       if (regexec(sed_cmd->sub_match, line, 0, NULL, 0) == 0) {
+               regmatch_t *regmatch = xmalloc(sizeof(regmatch_t) * (sed_cmd->num_backrefs+1));
+               int i;
+               char *ptr = (char *)line;
+
+               while (*ptr) {
+                       /* if we can match the search string... */
+                       if (regexec(sed_cmd->sub_match, ptr, sed_cmd->num_backrefs+1, regmatch, 0) == 0) {
+                               /* print everything before the match, */
+                               for (i = 0; i < regmatch[0].rm_so; i++) {
+#ifdef BB_FEATURE_SED_PATTERN_SPACE
+                                       if(!be_quiet || (sed_cmd->sub_flags & SUB_P))
+#endif
+                                       fputc(ptr[i], stdout);
+                               }
+
+                               /* then print the substitution in its place */
+#ifdef BB_FEATURE_SED_PATTERN_SPACE
+                               if(!be_quiet || (sed_cmd->sub_flags & SUB_P))
+#endif
+                               print_subst_w_backrefs(ptr, sed_cmd->replace, regmatch);
+
+                               /* then advance past the match */
+                               ptr += regmatch[0].rm_eo;
+
+                               /* and flag that something has changed */
+                               altered++;
+
+                               /* if we're not doing this globally... */
+                               if (!sed_cmd->sub_flags & SUB_G)
+                                       break;
+                       }
+                       /* if we COULD NOT match the search string (meaning we've gone past
+                        * all previous instances), get out */
+                       else
+                               break;
+               }
+
+               /* is there anything left to print? */
+#ifdef BB_FEATURE_SED_PATTERN_SPACE
+               if (*ptr && (!be_quiet || sed_cmds->sub_flags & SUB_P))
+#else
+               if (*ptr) 
+#endif
+                       fputs(ptr, stdout);
+
+               /* cleanup */
+               free(regmatch);
+       }
+
+       return altered;
+}
 
 static int do_sed_command(const struct sed_cmd *sed_cmd, const char *line) 
 {
@@ -355,43 +606,23 @@ static int do_sed_command(const struct sed_cmd *sed_cmd, const char *line)
                        altered++;
                        break;
 
-               case 's': /* oo, a fun one :-) */
-
-                       /* we only substitute if the substitution 'search' expression matches */
-                       if (regexec(sed_cmd->sub_match, line, 0, NULL, 0) == 0) {
-                               regmatch_t regmatch;
-                               int i;
-                               char *ptr = (char *)line;
-
-                               while (*ptr) {
-                                       /* if we can match the search string... */
-                                       if (regexec(sed_cmd->sub_match, ptr, 1, &regmatch, 0) == 0) {
-                                               /* print everything before the match, */
-                                               for (i = 0; i < regmatch.rm_so; i++)
-                                                       fputc(ptr[i], stdout);
-                                               /* then print the substitution in its place */
-                                               fputs(sed_cmd->replace, stdout);
-                                               /* then advance past the match */
-                                               ptr += regmatch.rm_eo;
-                                               /* and let the calling function know that something
-                                                * has been changed */
-                                               altered++;
-
-                                               /* if we're not doing this globally... */
-                                               if (!sed_cmd->sub_g)
-                                                       break;
-                                       }
-                                       /* if we COULD NOT match the search string (meaning we've gone past
-                                        * all previous instances), get out */
-                                       else
-                                               break;
-                               }
+               case 's':
+                       altered = do_subst_command(sed_cmd, line);
+                       break;
 
-                               /* is there anything left to print? */
-                               if (*ptr) 
-                                       fputs(ptr, stdout);
-                       }
+               case 'a':
+                       fputs(line, stdout);
+                       fputs(sed_cmd->editline, stdout);
+                       altered++;
+                       break;
 
+               case 'i':
+                       fputs(sed_cmd->editline, stdout);
+                       break;
+
+               case 'c':
+                       fputs(sed_cmd->editline, stdout);
+                       altered++;
                        break;
        }
 
@@ -432,8 +663,8 @@ static void process_file(FILE *file)
                        }
 
                        /* are we acting on a range of line numbers? */
-                       else if (sed_cmds[i].beg_line > 0 && sed_cmds[i].end_line > 0) {
-                               if (linenum >= sed_cmds[i].beg_line && linenum <= sed_cmds[i].end_line)
+                       else if (sed_cmds[i].beg_line > 0 && sed_cmds[i].end_line != 0) {
+                               if (linenum >= sed_cmds[i].beg_line && (sed_cmds[i].end_line == -1 || linenum <= sed_cmds[i].end_line))
                                        line_altered += do_sed_command(&sed_cmds[i], line);
                        }
 
@@ -449,9 +680,14 @@ static void process_file(FILE *file)
 
                }
 
-               /* we will print the line unless we were told to be quiet or if the
-                * line was altered (via a 'd'elete or 's'ubstitution) */
-               if (!be_quiet && !line_altered)
+               /* we will print the line unless we were told to be quiet or if
+                * the line was altered (via a 'd'elete or 's'ubstitution) */
+#ifndef BB_FEATURE_SED_PATTERN_SPACE
+               if (!be_quiet &&!line_altered)
+#else
+               /*  we where specificly requested to print the output */
+               if ((!be_quiet || (sed_cmds[i].sub_flags & SUB_P)) && !line_altered)
+#endif
                        fputs(line, stdout);
 
                free(line);
@@ -462,11 +698,7 @@ extern int sed_main(int argc, char **argv)
 {
        int opt;
 
-       /* do special-case option parsing */
-       if (argv[1] && (strcmp(argv[1], "--help") == 0))
-               usage(sed_usage);
-
-#if 0
+#ifdef BB_FEATURE_CLEAN_UP
        /* destroy command strings on exit */
        if (atexit(destroy_cmd_strs) == -1) {
                perror("sed");
@@ -475,12 +707,8 @@ extern int sed_main(int argc, char **argv)
 #endif
 
        /* do normal option parsing */
-       while ((opt = getopt(argc, argv, "Vhne:f:")) > 0) {
+       while ((opt = getopt(argc, argv, "hne:f:")) > 0) {
                switch (opt) {
-                       case 'V':
-                               printf("Print Busybox version here\n");
-                               exit(0);
-                               break;
                        case 'h':
                                usage(sed_usage);
                                break;
@@ -520,7 +748,7 @@ extern int sed_main(int argc, char **argv)
                for (i = optind; i < argc; i++) {
                        file = fopen(argv[i], "r");
                        if (file == NULL) {
-                               fprintf(stderr, "sed: %s: %s\n", argv[i], strerror(errno));
+                               errorMsg("%s: %s\n", argv[i], strerror(errno));
                        } else {
                                process_file(file);
                                fclose(file);