source: patches/grep-2.6.3-fixes-1.patch@ 6022b0e

grep-2.6.3

@@ -1 +1 @@
 GNU grep NEWS                                    -*- outline -*-
 
+  --mmap was meant to be ignored in 2.6.x, but it was instead
+  removed by mistake.  [bug introduced in 2.6]
+
 * Noteworthy changes in release 2.6.3 (2010-04-02) [stable]
 
 ** Bug fixes

configure

@@ -24372 +24372 @@
 if test "$ac_res" != no; then :
   test "$ac_res" = "none required" || LIBS="$ac_res $LIBS"
   test "$ac_cv_search_pcre_compile" = "none required" ||
-         LIB_PCRE=$ac_cv_search_pcre_compile
+         LIB="-ldl $LIBS"
 fi
 
       for ac_func in pcre_compile
 do :
   ac_fn_c_check_func "$LINENO" "pcre_compile" "ac_cv_func_pcre_compile"
 if test "x$ac_cv_func_pcre_compile" = x""yes; then :
+  CPPFLAGS="$CPPFLAGS -DHAVE_DYNAMIC_LIBPCRE"
   cat >>confdefs.h <<_ACEOF
 #define HAVE_PCRE_COMPILE 1
 _ACEOF

doc/grep.in.1

@@ -25 +25 @@
 .hy 0
 .
 .SH NAME
-grep, egrep, fgrep \- print lines matching a pattern
+grep, egrep, fgrep, rgrep \- print lines matching a pattern
 .
 .SH SYNOPSIS
 .B grep
@@ -56 +56 @@
 .B grep
 prints the matching lines.
 .PP
-In addition, two variant programs
-.B egrep
-and
+In addition, three variant programs
+.B egrep,
 .B fgrep
+and
+.B rgrep
 are available.
 .B egrep
 is the same as
@@ -67 +68 @@
 .B fgrep
 is the same as
 .BR "grep\ \-F" .
+.B rgrep
+is the same as
+.BR "grep\ \-r" .
 Direct invocation as either
 .B egrep
 or
@@ -825 +829 @@
 is not special if it would be the start of an invalid interval
 specification.
 For example, the command
-.B "grep\ \-E\ '{1'"
+.B "grep\ \-E\ \(aq{1\(aq"
 searches for the two-character string
 .B {1
 instead of reporting a syntax error in the regular expression.
@@ -865 +869 @@
 For example, if
 .B GREP_OPTIONS
 is
-.BR "'\-\^\-binary-files=without-match \-\^\-directories=skip'" ,
+.BR "\(aq\-\^\-binary-files=without-match \-\^\-directories=skip\(aq" ,
 .B grep
 behaves as if the two options
 .B \-\^\-binary\-files=without-match
@@ -1160 +1164 @@
 is not set.
 .
 .SH "EXIT STATUS"
-Normally, the exit status is 0 if selected lines are found and 1 otherwise.
-But the exit status is 2 if an error occurred, unless the
-.B \-q
-or
-.B \-\^\-quiet
-or
-.B \-\^\-silent
-option is used and a selected line is found.
-Note, however, that \s-1POSIX\s0 only mandates, for programs such as
-.BR grep ,
-.BR cmp ,
-and
-.BR diff ,
-that the exit status in case of error be greater than 1;
-it is therefore advisable, for the sake of portability,
-to use logic that tests for this general condition
-instead of strict equality with\ 2.
+The exit status is 0 if selected lines are found, and 1 if not found.          
+If an error occurred the exit status is 2.  (Note: POSIX error 
+handling code should check for '2' or greater.)
 .
 .SH COPYRIGHT
 Copyright 1998-2000, 2002, 2005-2010 Free Software Foundation, Inc.

grep-2.6.3

@@ -278 +278 @@
 #: src/main.c:1165
 #, c-format
 msgid "Binary file %s matches\n"
-msgstr "Concidència en el fitxer binari %s\n"
+msgstr "Coincidència en el fitxer binari %s\n"
 
 #: src/main.c:1179
 msgid "(standard input)"

src/kwset.c

@@ -40 +40 @@
 #ifdef GREP
 # include "xalloc.h"
 # undef malloc
-# define malloc(s) xmalloc(s)
+# define malloc xmalloc
 #endif
 
 #define NCHAR (UCHAR_MAX + 1)

src/main.c

@@ -266 +266 @@
 static char const short_options[] =
 "0123456789A:B:C:D:EFGHIPTUVX:abcd:e:f:hiKLlm:noqRrsuvwxyZz";
 
+/* Default for `file_list' if no files are given on the command line. */
+static char *stdin_argv[] =
+{
+  "-", NULL
+};
+
 /* Non-boolean long options that have no corresponding short equivalents.  */
 enum
 {
@@ -511 +517 @@
          for byte sentinels fore and aft.  */
       newalloc = newsize + pagesize + 1;
 
-      newbuf = bufalloc < newalloc ? xmalloc (bufalloc = newalloc) : buffer;
+      newbuf = bufalloc < newalloc ? malloc (bufalloc = newalloc) : buffer;
+      if (newbuf == NULL)
+        {
+          int saved_errno = errno;
+          free (buffer);
+          bufalloc = ALIGN_TO (INITIAL_BUFSIZE, pagesize) + pagesize + 1;
+          buffer = xmalloc (bufalloc);
+          errno = saved_errno;
+          return 0;
+        }
       readbuf = ALIGN_TO (newbuf + 1 + save, pagesize);
       bufbeg = readbuf - save;
       memmove (bufbeg, buffer + saved_offset, save);
@@ -1739 +1754 @@
   int opt, cc, status;
   int default_context;
   FILE *fp;
+  char **file_list;
 
   initialize_main (&argc, &argv);
   set_program_name (argv[0]);
@@ -2061 +2077 @@
         label = optarg;
         break;
 
+      case MMAP_OPTION:
       case 0:
         /* long options */
         break;
@@ -2154 +2171 @@
   if (max_count == 0)
     exit (EXIT_FAILURE);
 
-  if (optind < argc)
+  file_list = (optind == argc ? stdin_argv : &argv[optind]);
+
+  status = 1;
+  while (1)
     {
-        status = 1;
-        do
-        {
-          char *file = argv[optind];
-          if ((included_patterns || excluded_patterns)
-              && !isdir (file))
-            {
-              if (included_patterns &&
-                  ! excluded_file_name (included_patterns, file))
-                continue;
-              if (excluded_patterns &&
-                  excluded_file_name (excluded_patterns, file))
-                continue;
-            }
-          status &= grepfile (strcmp (file, "-") == 0 ? (char *) NULL : file,
-                              &stats_base);
-        }
-        while ( ++optind < argc);
+        char *file = *file_list++;
+
+        if (file == NULL)
+          break;
+
+        if ((included_patterns || excluded_patterns)
+           && !isdir (file))
+          {
+            if (included_patterns &&
+                ! excluded_file_name (included_patterns, file))
+              continue;
+            if (excluded_patterns &&
+                excluded_file_name (excluded_patterns, file))
+              continue;
+          }
+        status &= grepfile (strcmp (file, "-") == 0
+                           ? (char *) NULL : file, &stats_base);
     }
-  else
-    status = grepfile ((char *) NULL, &stats_base);
 
   /* We register via atexit() to test stdout.  */
   exit (errseen ? EXIT_TROUBLE : status);

src/pcresearch.c

@@ -22 +22 @@
 #include "search.h"
 #if HAVE_PCRE_H
 # include <pcre.h>
+# ifdef HAVE_DYNAMIC_LIBPCRE
+#  include <dlfcn.h>
+# endif
 #elif HAVE_PCRE_PCRE_H
 # include <pcre/pcre.h>
+# ifdef HAVE_DYNAMIC_LIBPCRE
+#  include <dlfcn.h>
+# endif
 #endif
 
 #if HAVE_LIBPCRE
@@ -34 +40 @@
 static pcre_extra *extra;
 #endif
 
+#ifdef HAVE_DYNAMIC_LIBPCRE
+
+# define pcre_compile dl_pcre_compile
+# define pcre_study dl_pcre_study
+# define pcre_exec dl_pcre_exec
+# define pcre_maketables dl_pcre_maketables
+
+static pcre *(*pcre_compile)(const char *pattern, int options,
+const char **errptr, int *erroffset,
+                               const unsigned char *tableptr);
+static pcre_extra *(*pcre_study)(const pcre *code, int options,
+                                   const char **errptr);
+static int (*pcre_exec)(const pcre *code, const pcre_extra *extra,
+                          const char *subject, int length, int startoffset,
+                          int options, int *ovector, int ovecsize);
+static const unsigned char *(*pcre_maketables)(void);
+
+static int
+map_pcre(void)
+{
+  void *library;
+
+  if (pcre_maketables)
+    return 1;
+
+  if (!(library = dlopen("libpcre.so.3", RTLD_NOW)))
+    return 0;
+
+  if (!(pcre_compile = dlsym(library, "pcre_compile")))
+    return 0;
+  if (!(pcre_study = dlsym(library, "pcre_study")))
+    return 0;
+  if (!(pcre_exec = dlsym(library, "pcre_exec")))
+    return 0;
+  if (!(pcre_maketables = dlsym(library, "pcre_maketables")))
+    return 0;
+
+  return 1;
+}
+
+#else
+#define map_pcre() (1)
+#endif /* HAVE_DYNAMIC_LIBPCRE */
+
 void
 Pcompile (char const *pattern, size_t size)
 {
@@ -51 +101 @@
   char const *p;
   char const *pnul;
 
+  if (!map_pcre ())
+    error (EXIT_TROUBLE, 0, "%s",
+           _("The -P option is not supported: libpcre.so.3 is not available"));
+
   /* FIXME: Remove these restrictions.  */
   if (memchr(pattern, '\n', size))
     error (EXIT_TROUBLE, 0, _("the -P option only supports a single pattern"));