Don't compare boolean values against TRUE or FALSE

[deliverable/binutils-gdb.git] / binutils / strings.c

diff --git a/binutils/strings.c b/binutils/strings.c
index 68c244cafbd816b578bd008108e5fe811342559a..13e261923b8876e480010b97eb37a7d46c2811b5 100644 (file)
--- a/binutils/strings.c
+++ b/binutils/strings.c
@@ -1,10 +1,9 @@
 /* strings -- print the strings of printable characters in files
 /* strings -- print the strings of printable characters in files

-   Copyright 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000, 2001,
-   2002, 2003 Free Software Foundation, Inc.
+   Copyright (C) 1993-2017 Free Software Foundation, Inc.

 
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
 
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by

-   the Free Software Foundation; either version 2, or (at your option)
+   the Free Software Foundation; either version 3, or (at your option)

    any later version.
 
    This program is distributed in the hope that it will be useful,
    any later version.
 
    This program is distributed in the hope that it will be useful,


@@ -14,15 +13,18 @@
 
    You should have received a copy of the GNU General Public License
    along with this program; if not, write to the Free Software
 
    You should have received a copy of the GNU General Public License
    along with this program; if not, write to the Free Software

-   Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA
-   02111-1307, USA.  */
+   Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston, MA
+   02110-1301, USA.  */

 \f
 /* Usage: strings [options] file...
 
    Options:
    --all
    -a
 \f
 /* Usage: strings [options] file...
 
    Options:
    --all
    -a

-   -           Do not scan only the initialized data section of object files.
+   -           Scan each file in its entirety.
+
+   --data
+   -d          Scan only the initialized data section(s) of object files.

 
    --print-file-name
    -f          Print the name of the file before each string.
 
    --print-file-name
    -f          Print the name of the file before each string.


@@ -36,6 +38,10 @@
    -t {o,x,d}  Print the offset within the file before each string,
                in octal/hex/decimal.
 
    -t {o,x,d}  Print the offset within the file before each string,
                in octal/hex/decimal.
 

+  --include-all-whitespace
+  -w           By default tab and space are the only whitepace included in graphic
+               char sequences.  This option considers all of isspace() valid.
+

    -o          Like -to.  (Some other implementations have -o like -to,
                others like -td.  We chose one arbitrarily.)
 
    -o          Like -to.  (Some other implementations have -o like -to,
                others like -td.  We chose one arbitrarily.)
 


@@ -46,49 +52,36 @@
                littleendian 32-bit.
 
    --target=BFDNAME
                littleendian 32-bit.
 
    --target=BFDNAME

+   -T {bfdname}

                Specify a non-default object file format.
 
                Specify a non-default object file format.
 

+  --output-separator=sep_string
+  -s sep_string        String used to separate parsed strings in output.
+               Default is newline.
+

    --help
    -h          Print the usage message on the standard output.
 
    --version
    --help
    -h          Print the usage message on the standard output.
 
    --version

+   -V

    -v          Print the program version number.
 
    Written by Richard Stallman <rms@gnu.ai.mit.edu>
    and David MacKenzie <djm@gnu.ai.mit.edu>.  */
 
    -v          Print the program version number.
 
    Written by Richard Stallman <rms@gnu.ai.mit.edu>
    and David MacKenzie <djm@gnu.ai.mit.edu>.  */
 

-#ifdef HAVE_CONFIG_H
-#include "config.h"
-#endif
+#include "sysdep.h"

 #include "bfd.h"
 #include "bfd.h"

-#include <stdio.h>

 #include "getopt.h"
 #include "getopt.h"

-#include <errno.h>
-#include "bucomm.h"

 #include "libiberty.h"
 #include "safe-ctype.h"
 #include "libiberty.h"
 #include "safe-ctype.h"

-
-/* Some platforms need to put stdin into binary mode, to read
-    binary files.  */
-#ifdef HAVE_SETMODE
-#ifndef O_BINARY
-#ifdef _O_BINARY
-#define O_BINARY _O_BINARY
-#define setmode _setmode
-#else
-#define O_BINARY 0
-#endif
-#endif
-#if O_BINARY
-#include <io.h>
-#define SET_BINARY(f) do { if (!isatty (f)) setmode (f,O_BINARY); } while (0)
-#endif
-#endif
+#include "bucomm.h"

 
 #define STRING_ISGRAPHIC(c) \
       (   (c) >= 0 \
        && (c) <= 255 \
 
 #define STRING_ISGRAPHIC(c) \
       (   (c) >= 0 \
        && (c) <= 255 \

-       && ((c) == '\t' || ISPRINT (c) || (encoding == 'S' && (c) > 127)))
+       && ((c) == '\t' || ISPRINT (c) || (encoding == 'S' && (c) > 127) \
+          || (include_all_whitespace && ISSPACE (c))) \
+      )

 
 #ifndef errno
 extern int errno;
 
 #ifndef errno
 extern int errno;


@@ -97,20 +90,15 @@ extern int errno;
 /* The BFD section flags that identify an initialized data section.  */
 #define DATA_FLAGS (SEC_ALLOC | SEC_LOAD | SEC_HAS_CONTENTS)
 
 /* The BFD section flags that identify an initialized data section.  */
 #define DATA_FLAGS (SEC_ALLOC | SEC_LOAD | SEC_HAS_CONTENTS)
 

-#ifdef HAVE_FOPEN64
-typedef off64_t file_off;
-#define file_open(s,m) fopen64(s, m)
-#else
-typedef off_t file_off;
-#define file_open(s,m) fopen(s, m)
-#endif
-

 /* Radix for printing addresses (must be 8, 10 or 16).  */
 static int address_radix;
 
 /* Minimum length of sequence of graphic chars to trigger output.  */
 static int string_min;
 
 /* Radix for printing addresses (must be 8, 10 or 16).  */
 static int address_radix;
 
 /* Minimum length of sequence of graphic chars to trigger output.  */
 static int string_min;
 

+/* Whether or not we include all whitespace as a graphic char.   */
+static bfd_boolean include_all_whitespace;
+

 /* TRUE means print address within file for each string.  */
 static bfd_boolean print_addresses;
 
 /* TRUE means print address within file for each string.  */
 static bfd_boolean print_addresses;
 


@@ -130,26 +118,37 @@ static char *target;
 static char encoding;
 static int encoding_bytes;
 
 static char encoding;
 static int encoding_bytes;
 

+/* Output string used to separate parsed strings  */
+static char *output_separator;
+

 static struct option long_options[] =
 {
   {"all", no_argument, NULL, 'a'},
 static struct option long_options[] =
 {
   {"all", no_argument, NULL, 'a'},

+  {"data", no_argument, NULL, 'd'},

   {"print-file-name", no_argument, NULL, 'f'},
   {"bytes", required_argument, NULL, 'n'},
   {"radix", required_argument, NULL, 't'},
   {"print-file-name", no_argument, NULL, 'f'},
   {"bytes", required_argument, NULL, 'n'},
   {"radix", required_argument, NULL, 't'},

+  {"include-all-whitespace", required_argument, NULL, 'w'},

   {"encoding", required_argument, NULL, 'e'},
   {"target", required_argument, NULL, 'T'},
   {"encoding", required_argument, NULL, 'e'},
   {"target", required_argument, NULL, 'T'},

+  {"output-separator", required_argument, NULL, 's'},

   {"help", no_argument, NULL, 'h'},
   {"version", no_argument, NULL, 'v'},
   {NULL, 0, NULL, 0}
 };
 
   {"help", no_argument, NULL, 'h'},
   {"version", no_argument, NULL, 'v'},
   {NULL, 0, NULL, 0}
 };
 

-static void strings_a_section (bfd *, asection *, void *);
-static bfd_boolean strings_object_file (const char *);
-static bfd_boolean strings_file (char *file);
-static int integer_arg (char *s);
-static void print_strings (const char *, FILE *, file_off, int, int, char *);
-static void usage (FILE *, int);
-static long get_char (FILE *, file_off *, int *, char **);
+/* Records the size of a named file so that we
+   do not repeatedly run bfd_stat() on it.  */
+
+typedef struct
+{
+  const char *  filename;
+  bfd_size_type filesize;
+} filename_and_size_t;
+
+static bfd_boolean strings_file (char *);
+static void print_strings (const char *, FILE *, file_ptr, int, int, char *);
+static void usage (FILE *, int) ATTRIBUTE_NORETURN;

 \f
 int main (int, char **);
 
 \f
 int main (int, char **);
 


@@ -159,6 +158,8 @@ main (int argc, char **argv)
   int optc;
   int exit_status = 0;
   bfd_boolean files_given = FALSE;
   int optc;
   int exit_status = 0;
   bfd_boolean files_given = FALSE;

+  char *s;
+  int numeric_opt = 0;

 
 #if defined (HAVE_SETLOCALE)
   setlocale (LC_ALL, "");
 
 #if defined (HAVE_SETLOCALE)
   setlocale (LC_ALL, "");


@@ -168,14 +169,23 @@ main (int argc, char **argv)
 
   program_name = argv[0];
   xmalloc_set_program_name (program_name);
 
   program_name = argv[0];
   xmalloc_set_program_name (program_name);

-  string_min = -1;
+  bfd_set_error_program_name (program_name);
+
+  expandargv (&argc, &argv);
+
+  string_min = 4;
+  include_all_whitespace = FALSE;

   print_addresses = FALSE;
   print_filenames = FALSE;
   print_addresses = FALSE;
   print_filenames = FALSE;

-  datasection_only = TRUE;
+  if (DEFAULT_STRINGS_ALL)
+    datasection_only = FALSE;
+  else
+    datasection_only = TRUE;

   target = NULL;
   encoding = 's';
   target = NULL;
   encoding = 's';

+  output_separator = NULL;

 
 

-  while ((optc = getopt_long (argc, argv, "afhHn:ot:e:Vv0123456789",
+  while ((optc = getopt_long (argc, argv, "adfhHn:wot:e:T:s:Vv0123456789",

                              long_options, (int *) 0)) != EOF)
     {
       switch (optc)
                              long_options, (int *) 0)) != EOF)
     {
       switch (optc)


@@ -184,6 +194,10 @@ main (int argc, char **argv)
          datasection_only = FALSE;
          break;
 
          datasection_only = FALSE;
          break;
 

+       case 'd':
+         datasection_only = TRUE;
+         break;
+

        case 'f':
          print_filenames = TRUE;
          break;
        case 'f':
          print_filenames = TRUE;
          break;


@@ -193,9 +207,13 @@ main (int argc, char **argv)
          usage (stdout, 0);
 
        case 'n':
          usage (stdout, 0);
 
        case 'n':

-         string_min = integer_arg (optarg);
-         if (string_min < 1)
-           fatal (_("invalid number %s"), optarg);
+         string_min = (int) strtoul (optarg, &s, 0);
+         if (s != NULL && *s != 0)
+           fatal (_("invalid integer argument %s"), optarg);
+         break;
+
+       case 'w':
+         include_all_whitespace = TRUE;

          break;
 
        case 'o':
          break;
 
        case 'o':


@@ -236,6 +254,10 @@ main (int argc, char **argv)
          encoding = optarg[0];
          break;
 
          encoding = optarg[0];
          break;
 

+       case 's':
+         output_separator = optarg;
+          break;
+

        case 'V':
        case 'v':
          print_version ("strings");
        case 'V':
        case 'v':
          print_version ("strings");


@@ -245,16 +267,19 @@ main (int argc, char **argv)
          usage (stderr, 1);
 
        default:
          usage (stderr, 1);
 
        default:

-         if (string_min < 0)
-           string_min = optc - '0';
-         else
-           string_min = string_min * 10 + optc - '0';
+         numeric_opt = optind;

          break;
        }
     }
 
          break;
        }
     }
 

-  if (string_min < 0)
-    string_min = 4;
+  if (numeric_opt != 0)
+    {
+      string_min = (int) strtoul (argv[numeric_opt - 1] + 1, &s, 0);
+      if (s != NULL && *s != 0)
+       fatal (_("invalid integer argument %s"), argv[numeric_opt - 1] + 1);
+    }
+  if (string_min < 1)
+    fatal (_("invalid minimum string length %d"), string_min);

 
   switch (encoding)
     {
 
   switch (encoding)
     {


@@ -280,9 +305,7 @@ main (int argc, char **argv)
   if (optind >= argc)
     {
       datasection_only = FALSE;
   if (optind >= argc)
     {
       datasection_only = FALSE;

-#ifdef SET_BINARY

       SET_BINARY (fileno (stdin));
       SET_BINARY (fileno (stdin));

-#endif

       print_strings ("{standard input}", stdin, 0, 0, 0, (char *) NULL);
       files_given = TRUE;
     }
       print_strings ("{standard input}", stdin, 0, 0, 0, (char *) NULL);
       files_given = TRUE;
     }


@@ -295,7 +318,7 @@ main (int argc, char **argv)
          else
            {
              files_given = TRUE;
          else
            {
              files_given = TRUE;

-             exit_status |= strings_file (argv[optind]) == FALSE;
+             exit_status |= !strings_file (argv[optind]);

            }
        }
     }
            }
        }
     }


@@ -306,27 +329,62 @@ main (int argc, char **argv)
   return (exit_status);
 }
 \f
   return (exit_status);
 }
 \f

-/* Scan section SECT of the file ABFD, whose printable name is FILE.
-   If it contains initialized data,
-   set `got_a_section' and print the strings in it.  */
+/* Scan section SECT of the file ABFD, whose printable name is in
+   ARG->filename and whose size might be in ARG->filesize.  If it
+   contains initialized data set `got_a_section' and print the
+   strings in it.
+
+   FIXME: We ought to be able to return error codes/messages for
+   certain conditions.  */

 
 static void
 
 static void

-strings_a_section (bfd *abfd, asection *sect, void *filearg)
+strings_a_section (bfd *abfd, asection *sect, void *arg)

 {
 {

-  const char *file = (const char *) filearg;
+  filename_and_size_t * filename_and_sizep;
+  bfd_size_type *filesizep;
+  bfd_size_type sectsize;
+  void *mem;
+
+  if ((sect->flags & DATA_FLAGS) != DATA_FLAGS)
+    return;
+
+  sectsize = bfd_get_section_size (sect);
+
+  if (sectsize <= 0)
+    return;

 
 

-  if ((sect->flags & DATA_FLAGS) == DATA_FLAGS)
+  /* Get the size of the file.  This might have been cached for us.  */
+  filename_and_sizep = (filename_and_size_t *) arg;
+  filesizep = & filename_and_sizep->filesize;
+
+  if (*filesizep == 0)

     {
     {

-      bfd_size_type sz = bfd_get_section_size_before_reloc (sect);
-      void *mem = xmalloc (sz);
+      struct stat st;

 
 

-      if (bfd_get_section_contents (abfd, sect, mem, (file_ptr) 0, sz))
-       {
-         got_a_section = TRUE;
-         print_strings (file, (FILE *) NULL, sect->filepos, 0, sz, mem);
-       }
-      free (mem);
+      if (bfd_stat (abfd, &st))
+       return;
+
+      /* Cache the result so that we do not repeatedly stat this file.  */
+      *filesizep = st.st_size;
+    }
+
+  /* Compare the size of the section against the size of the file.
+     If the section is bigger then the file must be corrupt and
+     we should not try dumping it.  */
+  if (sectsize >= *filesizep)
+    return;
+
+  mem = xmalloc (sectsize);
+
+  if (bfd_get_section_contents (abfd, sect, mem, (file_ptr) 0, sectsize))
+    {
+      got_a_section = TRUE;
+
+      print_strings (filename_and_sizep->filename, NULL, sect->filepos,
+                    0, sectsize, (char *) mem);

     }
     }

+
+  free (mem);

 }
 
 /* Scan all of the sections in FILE, and print the strings
 }
 
 /* Scan all of the sections in FILE, and print the strings


@@ -338,7 +396,10 @@ strings_a_section (bfd *abfd, asection *sect, void *filearg)
 static bfd_boolean
 strings_object_file (const char *file)
 {
 static bfd_boolean
 strings_object_file (const char *file)
 {

-  bfd *abfd = bfd_openr (file, target);
+  filename_and_size_t filename_and_size;
+  bfd *abfd;
+
+  abfd = bfd_openr (file, target);

 
   if (abfd == NULL)
     /* Treat the file as a non-object file.  */
 
   if (abfd == NULL)
     /* Treat the file as a non-object file.  */


@@ -354,7 +415,9 @@ strings_object_file (const char *file)
     }
 
   got_a_section = FALSE;
     }
 
   got_a_section = FALSE;

-  bfd_map_over_sections (abfd, strings_a_section, (void *) file);
+  filename_and_size.filename = file;
+  filename_and_size.filesize = 0;
+  bfd_map_over_sections (abfd, strings_a_section, & filename_and_size);

 
   if (!bfd_close (abfd))
     {
 
   if (!bfd_close (abfd))
     {


@@ -370,8 +433,19 @@ strings_object_file (const char *file)
 static bfd_boolean
 strings_file (char *file)
 {
 static bfd_boolean
 strings_file (char *file)
 {

-  if (get_file_size (file) < 1)
-    return FALSE;
+  struct stat st;
+
+  /* get_file_size does not support non-S_ISREG files.  */
+
+  if (stat (file, &st) < 0)
+    {
+      if (errno == ENOENT)
+       non_fatal (_("'%s': No such file"), file);
+      else
+       non_fatal (_("Warning: could not locate '%s'.  reason: %s"),
+                  file, strerror (errno));
+      return FALSE;
+    }

 
   /* If we weren't told to scan the whole file,
      try to open it as an object file and only look at
 
   /* If we weren't told to scan the whole file,
      try to open it as an object file and only look at


@@ -381,7 +455,7 @@ strings_file (char *file)
     {
       FILE *stream;
 
     {
       FILE *stream;
 

-      stream = file_open (file, FOPEN_RB);
+      stream = fopen (file, FOPEN_RB);

       if (stream == NULL)
        {
          fprintf (stderr, "%s: ", program_name);
       if (stream == NULL)
        {
          fprintf (stderr, "%s: ", program_name);


@@ -389,7 +463,7 @@ strings_file (char *file)
          return FALSE;
        }
 
          return FALSE;
        }
 

-      print_strings (file, stream, (file_off) 0, 0, 0, (char *) 0);
+      print_strings (file, stream, (file_ptr) 0, 0, 0, (char *) 0);

 
       if (fclose (stream) == EOF)
        {
 
       if (fclose (stream) == EOF)
        {


@@ -413,11 +487,10 @@ strings_file (char *file)
    MAGICCOUNT is how many characters are in it.  */
 
 static long
    MAGICCOUNT is how many characters are in it.  */
 
 static long

-get_char (FILE *stream, file_off *address, int *magiccount, char **magic)
+get_char (FILE *stream, file_ptr *address, int *magiccount, char **magic)

 {
   int c, i;
 {
   int c, i;

-  long r = EOF;
-  unsigned char buf[4];
+  long r = 0;

 
   for (i = 0; i < encoding_bytes; i++)
     {
 
   for (i = 0; i < encoding_bytes; i++)
     {


@@ -430,7 +503,12 @@ get_char (FILE *stream, file_off *address, int *magiccount, char **magic)
        {
          if (stream == NULL)
            return EOF;
        {
          if (stream == NULL)
            return EOF;

-#ifdef HAVE_GETC_UNLOCKED
+
+         /* Only use getc_unlocked if we found a declaration for it.
+            Otherwise, libc is not thread safe by default, and we
+            should not use it.  */
+
+#if defined(HAVE_GETC_UNLOCKED) && HAVE_DECL_GETC_UNLOCKED

          c = getc_unlocked (stream);
 #else
          c = getc (stream);
          c = getc_unlocked (stream);
 #else
          c = getc (stream);


@@ -440,34 +518,22 @@ get_char (FILE *stream, file_off *address, int *magiccount, char **magic)
        }
 
       (*address)++;
        }
 
       (*address)++;

-      buf[i] = c;
+      r = (r << 8) | (c & 0xff);

     }
 
   switch (encoding)
     {
     }
 
   switch (encoding)
     {

-    case 'S':
-    case 's':
-      r = buf[0];
-      break;
-    case 'b':
-      r = (buf[0] << 8) | buf[1];
+    default:

       break;
     case 'l':
       break;
     case 'l':

-      r = buf[0] | (buf[1] << 8);
-      break;
-    case 'B':
-      r = ((long) buf[0] << 24) | ((long) buf[1] << 16) |
-       ((long) buf[2] << 8) | buf[3];
+      r = ((r & 0xff) << 8) | ((r & 0xff00) >> 8);

       break;
     case 'L':
       break;
     case 'L':

-      r = buf[0] | ((long) buf[1] << 8) | ((long) buf[2] << 16) |
-       ((long) buf[3] << 24);
+      r = (((r & 0xff) << 24) | ((r & 0xff00) << 8)
+          | ((r & 0xff0000) >> 8) | ((r & 0xff000000) >> 24));

       break;
     }
 
       break;
     }
 

-  if (r == EOF)
-    return 0;
-

   return r;
 }
 \f
   return r;
 }
 \f


@@ -484,14 +550,14 @@ get_char (FILE *stream, file_off *address, int *magiccount, char **magic)
    Those characters come at address ADDRESS and the data in STREAM follow.  */
 
 static void
    Those characters come at address ADDRESS and the data in STREAM follow.  */
 
 static void

-print_strings (const char *filename, FILE *stream, file_off address,
+print_strings (const char *filename, FILE *stream, file_ptr address,

               int stop_point, int magiccount, char *magic)
 {
   char *buf = (char *) xmalloc (sizeof (char) * (string_min + 1));
 
   while (1)
     {
               int stop_point, int magiccount, char *magic)
 {
   char *buf = (char *) xmalloc (sizeof (char) * (string_min + 1));
 
   while (1)
     {

-      file_off start;
+      file_ptr start;

       int i;
       long c;
 
       int i;
       long c;
 


@@ -504,7 +570,10 @@ print_strings (const char *filename, FILE *stream, file_off address,
        {
          c = get_char (stream, &address, &magiccount, &magic);
          if (c == EOF)
        {
          c = get_char (stream, &address, &magiccount, &magic);
          if (c == EOF)

-           return;
+           {
+             free (buf);
+             return;
+           }

          if (! STRING_ISGRAPHIC (c))
            /* Found a non-graphic.  Try again starting with next char.  */
            goto tryline;
          if (! STRING_ISGRAPHIC (c))
            /* Found a non-graphic.  Try again starting with next char.  */
            goto tryline;


@@ -512,7 +581,7 @@ print_strings (const char *filename, FILE *stream, file_off address,
        }
 
       /* We found a run of `string_min' graphic characters.  Print up
        }
 
       /* We found a run of `string_min' graphic characters.  Print up

-         to the next non-graphic character.  */
+        to the next non-graphic character.  */

 
       if (print_filenames)
        printf ("%s: ", filename);
 
       if (print_filenames)
        printf ("%s: ", filename);


@@ -520,46 +589,59 @@ print_strings (const char *filename, FILE *stream, file_off address,
        switch (address_radix)
          {
          case 8:
        switch (address_radix)
          {
          case 8:

-#if __STDC_VERSION__ >= 199901L || (defined(__GNUC__) && __GNUC__ >= 2)
+#ifdef HAVE_LONG_LONG

            if (sizeof (start) > sizeof (long))
            if (sizeof (start) > sizeof (long))

-             printf ("%7Lo ", (unsigned long long) start);
+             {
+# ifndef __MSVCRT__
+               printf ("%7llo ", (unsigned long long) start);
+# else
+               printf ("%7I64o ", (unsigned long long) start);
+# endif
+             }

            else
            else

-#else
-# if !BFD_HOST_64BIT_LONG
+#elif !BFD_HOST_64BIT_LONG

            if (start != (unsigned long) start)
              printf ("++%7lo ", (unsigned long) start);
            else
            if (start != (unsigned long) start)
              printf ("++%7lo ", (unsigned long) start);
            else

-# endif

 #endif
              printf ("%7lo ", (unsigned long) start);
            break;
 
          case 10:
 #endif
              printf ("%7lo ", (unsigned long) start);
            break;
 
          case 10:

-#if __STDC_VERSION__ >= 199901L || (defined(__GNUC__) && __GNUC__ >= 2)
+#ifdef HAVE_LONG_LONG

            if (sizeof (start) > sizeof (long))
            if (sizeof (start) > sizeof (long))

-             printf ("%7Ld ", (unsigned long long) start);
+             {
+# ifndef __MSVCRT__
+               printf ("%7lld ", (unsigned long long) start);
+# else
+               printf ("%7I64d ", (unsigned long long) start);
+# endif
+             }

            else
            else

-#else
-# if !BFD_HOST_64BIT_LONG
+#elif !BFD_HOST_64BIT_LONG

            if (start != (unsigned long) start)
              printf ("++%7ld ", (unsigned long) start);
            else
            if (start != (unsigned long) start)
              printf ("++%7ld ", (unsigned long) start);
            else

-# endif

 #endif
              printf ("%7ld ", (long) start);
            break;
 
          case 16:
 #endif
              printf ("%7ld ", (long) start);
            break;
 
          case 16:

-#if __STDC_VERSION__ >= 199901L || (defined(__GNUC__) && __GNUC__ >= 2)
+#ifdef HAVE_LONG_LONG

            if (sizeof (start) > sizeof (long))
            if (sizeof (start) > sizeof (long))

-             printf ("%7Lx ", (unsigned long long) start);
+             {
+# ifndef __MSVCRT__
+               printf ("%7llx ", (unsigned long long) start);
+# else
+               printf ("%7I64x ", (unsigned long long) start);
+# endif
+             }

            else
            else

-#else
-# if !BFD_HOST_64BIT_LONG
+#elif !BFD_HOST_64BIT_LONG

            if (start != (unsigned long) start)
            if (start != (unsigned long) start)

-             printf ("%lx%8.8lx ", start >> 32, start & 0xffffffff);
+             printf ("%lx%8.8lx ", (unsigned long) (start >> 32),
+                     (unsigned long) (start & 0xffffffff));

            else
            else

-# endif

 #endif
              printf ("%7lx ", (unsigned long) start);
            break;
 #endif
              printf ("%7lx ", (unsigned long) start);
            break;


@@ -578,74 +660,46 @@ print_strings (const char *filename, FILE *stream, file_off address,
          putchar (c);
        }
 
          putchar (c);
        }
 

-      putchar ('\n');
-    }
-}
-\f
-/* Parse string S as an integer, using decimal radix by default,
-   but allowing octal and hex numbers as in C.  */
-
-static int
-integer_arg (char *s)
-{
-  int value;
-  int radix = 10;
-  char *p = s;
-  int c;
-
-  if (*p != '0')
-    radix = 10;
-  else if (*++p == 'x')
-    {
-      radix = 16;
-      p++;
-    }
-  else
-    radix = 8;
-
-  value = 0;
-  while (((c = *p++) >= '0' && c <= '9')
-        || (radix == 16 && (c & ~40) >= 'A' && (c & ~40) <= 'Z'))
-    {
-      value *= radix;
-      if (c >= '0' && c <= '9')
-       value += c - '0';
+      if (output_separator)
+        fputs (output_separator, stdout);

       else
       else

-       value += (c & ~40) - 'A';
+        putchar ('\n');

     }
     }

-
-  if (c == 'b')
-    value *= 512;
-  else if (c == 'B')
-    value *= 1024;
-  else
-    p--;
-
-  if (*p)
-    fatal (_("invalid integer argument %s"), s);
-
-  return value;
+  free (buf);

 }
 }

-
+\f

 static void
 usage (FILE *stream, int status)
 {
   fprintf (stream, _("Usage: %s [option(s)] [file(s)]\n"), program_name);
   fprintf (stream, _(" Display printable strings in [file(s)] (stdin by default)\n"));
 static void
 usage (FILE *stream, int status)
 {
   fprintf (stream, _("Usage: %s [option(s)] [file(s)]\n"), program_name);
   fprintf (stream, _(" Display printable strings in [file(s)] (stdin by default)\n"));

-  fprintf (stream, _(" The options are:\n\
+  fprintf (stream, _(" The options are:\n"));
+
+  if (DEFAULT_STRINGS_ALL)
+    fprintf (stream, _("\
+  -a - --all                Scan the entire file, not just the data section [default]\n\
+  -d --data                 Only scan the data sections in the file\n"));
+  else
+    fprintf (stream, _("\

   -a - --all                Scan the entire file, not just the data section\n\
   -a - --all                Scan the entire file, not just the data section\n\

+  -d --data                 Only scan the data sections in the file [default]\n"));
+
+  fprintf (stream, _("\

   -f --print-file-name      Print the name of the file before each string\n\
   -n --bytes=[number]       Locate & print any NUL-terminated sequence of at\n\
   -f --print-file-name      Print the name of the file before each string\n\
   -n --bytes=[number]       Locate & print any NUL-terminated sequence of at\n\

-  -<number>                 least [number] characters (default 4).\n\
-  -t --radix={o,x,d}        Print the location of the string in base 8, 10 or 16\n\
+  -<number>                   least [number] characters (default 4).\n\
+  -t --radix={o,d,x}        Print the location of the string in base 8, 10 or 16\n\
+  -w --include-all-whitespace Include all whitespace as valid string characters\n\

   -o                        An alias for --radix=o\n\
   -T --target=<BFDNAME>     Specify the binary file format\n\
   -e --encoding={s,S,b,l,B,L} Select character size and endianness:\n\
                             s = 7-bit, S = 8-bit, {b,l} = 16-bit, {B,L} = 32-bit\n\
   -o                        An alias for --radix=o\n\
   -T --target=<BFDNAME>     Specify the binary file format\n\
   -e --encoding={s,S,b,l,B,L} Select character size and endianness:\n\
                             s = 7-bit, S = 8-bit, {b,l} = 16-bit, {B,L} = 32-bit\n\

+  -s --output-separator=<string> String used to separate strings in output.\n\
+  @<file>                   Read options from <file>\n\

   -h --help                 Display this information\n\
   -h --help                 Display this information\n\

-  -v --version              Print the program's version number\n"));
+  -v -V --version           Print the program's version number\n"));

   list_supported_targets (program_name, stream);
   list_supported_targets (program_name, stream);

-  if (status == 0)
+  if (REPORT_BUGS_TO[0] && status == 0)

     fprintf (stream, _("Report bugs to %s\n"), REPORT_BUGS_TO);
   exit (status);
 }
     fprintf (stream, _("Report bugs to %s\n"), REPORT_BUGS_TO);
   exit (status);
 }