Fix invocation of stat() on a NULL pointer.
authorNick Clifton <nickc@redhat.com>
Wed, 26 Apr 2017 14:42:03 +0000 (15:42 +0100)
committerNick Clifton <nickc@redhat.com>
Wed, 26 Apr 2017 14:42:03 +0000 (15:42 +0100)
PR binutils/21407
* bucomm.c (get_file_size): Return -1 if file_name is NULL.
* ar.c (main): Fail with usage() invocation if no file names are
provided.

binutils/ChangeLog
binutils/ar.c
binutils/bucomm.c

index e1736b9..b37d5b3 100644 (file)
@@ -1,5 +1,12 @@
 2017-04-26  Nick Clifton  <nickc@redhat.com>
 
+       PR binutils/21407
+       * bucomm.c (get_file_size): Return -1 if file_name is NULL.
+       * ar.c (main): Fail with usage() invocation if no file names are
+       provided.
+
+2017-04-26  Nick Clifton  <nickc@redhat.com>
+
        * readelf.c (process_section_headers): Warn about overlarge
        sections.
        (print_gnu_build_attribute_name): Print the number of unrecognised
index 32ac404..ef8b5bd 100644 (file)
@@ -738,6 +738,9 @@ main (int argc, char **argv)
 
   arg_index = 0;
 
+  if (argv[arg_index] == NULL)
+    usage (0);
+
   if (mri_mode)
     {
       default_deterministic ();
index e682717..fd6f356 100644 (file)
@@ -587,6 +587,9 @@ get_file_size (const char * file_name)
 {
   struct stat statbuf;
 
+  if (file_name == NULL)
+    return (off_t) -1;
+
   if (stat (file_name, &statbuf) < 0)
     {
       if (errno == ENOENT)