From: Nick Clifton <nickc@redhat.com>
Date: Wed, 26 Apr 2017 14:42:03 +0000 (+0100)
Subject: Fix invocation of stat() on a NULL pointer.
X-Git-Url: http://git.efficios.com/?a=commitdiff_plain;h=740a463062bd5d1641bdfb639295dafe89341b9b;p=deliverable%2Fbinutils-gdb.git

Fix invocation of stat() on a NULL pointer.

	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.
---

diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index e1736b9926..b37d5b3dcb 100644
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -1,3 +1,10 @@
+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
diff --git a/binutils/ar.c b/binutils/ar.c
index 32ac4040c6..ef8b5bdb12 100644
--- a/binutils/ar.c
+++ b/binutils/ar.c
@@ -738,6 +738,9 @@ main (int argc, char **argv)
 
   arg_index = 0;
 
+  if (argv[arg_index] == NULL)
+    usage (0);
+
   if (mri_mode)
     {
       default_deterministic ();
diff --git a/binutils/bucomm.c b/binutils/bucomm.c
index e682717b8d..fd6f35604d 100644
--- a/binutils/bucomm.c
+++ b/binutils/bucomm.c
@@ -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)