Fixed segfault with 'cut -f 1 -d:' and added 'cut -s' suport.

[oweals/busybox.git] / grep.c

diff --git a/grep.c b/grep.c
index 6872ac271018987890685ca8a5d92eb3efcc7cde..0e495ff87e19648c674941a47e80cd144839a568 100644 (file)
--- a/grep.c
+++ b/grep.c
@@ -39,6 +39,9 @@
 #include <signal.h>
 #include <time.h>
 #include <ctype.h>
+#define BB_DECLARE_EXTERN
+#define bb_need_too_few_args
+#include "messages.c"
 
 static const char grep_usage[] =
        "grep [OPTIONS]... PATTERN [FILE]...\n"
@@ -51,9 +54,9 @@ static const char grep_usage[] =
        "\t-q\tbe quiet. Returns 0 if result was found, 1 otherwise\n"
        "\t-v\tselect non-matching lines\n\n"
 #if defined BB_REGEXP
-       "This version of grep matches full regular expresions.\n";
+       "This version of grep matches full regular expressions.\n";
 #else
-       "This version of grep matches strings (not regular expresions).\n"
+       "This version of grep matches strings (not regular expressions).\n"
 #endif
 #endif
        ;
@@ -63,18 +66,12 @@ static int match = FALSE, beQuiet = FALSE;
 static void do_grep(FILE * fp, char *needle, char *fileName, int tellName,
                                        int ignoreCase, int tellLine, int invertSearch)
 {
-       char *cp;
        long line = 0;
-       char haystack[BUF_SIZE];
+       char *haystack;
        int  truth = !invertSearch;
 
-       while (fgets(haystack, sizeof(haystack), fp)) {
+       while ((haystack = cstring_lineFromFile(fp))) {
                line++;
-               cp = &haystack[strlen(haystack) - 1];
-
-               if (*cp != '\n')
-                       fprintf(stderr, "%s: Line too long\n", fileName);
-
                if (find_match(haystack, needle, ignoreCase) == truth) {
                        if (tellName == TRUE)
                                printf("%s:", fileName);
@@ -87,6 +84,7 @@ static void do_grep(FILE * fp, char *needle, char *fileName, int tellName,
 
                        match = TRUE;
                }
+               free(haystack);
        }
 }
 
@@ -94,7 +92,6 @@ static void do_grep(FILE * fp, char *needle, char *fileName, int tellName,
 extern int grep_main(int argc, char **argv)
 {
        FILE *fp;
-       char *cp;
        char *needle;
        char *fileName;
        int tellName     = TRUE;
@@ -102,18 +99,14 @@ extern int grep_main(int argc, char **argv)
        int tellLine     = FALSE;
        int invertSearch = FALSE;
 
-       argc--;
-       argv++;
        if (argc < 1) {
                usage(grep_usage);
        }
+       argv++;
 
-       if (**argv == '-') {
-               argc--;
-               cp = *argv++;
-
-               while (*++cp)
-                       switch (*cp) {
+       while (--argc >= 0 && *argv && (**argv == '-')) {
+               while (*++(*argv)) {
+                       switch (**argv) {
                        case 'i':
                                ignoreCase = TRUE;
                                break;
@@ -137,6 +130,12 @@ extern int grep_main(int argc, char **argv)
                        default:
                                usage(grep_usage);
                        }
+               }
+               argv++;
+       }
+
+       if (argc == 0 || *argv == NULL) {
+               fatalError(too_few_args, "grep");
        }
 
        needle = *argv++;
@@ -164,7 +163,7 @@ extern int grep_main(int argc, char **argv)
                        fclose(fp);
                }
        }
-       exit(match);
+       return(match);
 }