From: Tony Reix <tony.reix@atos.net>
Date: Fri, 28 Jul 2017 18:52:13 +0000 (+0000)
Subject: * xcoff.c: Don't leak a file descriptor if an archive is malformed.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=afce7debdc589469bd025d5c65697598c6ef325f;p=gcc.git

* xcoff.c: Don't leak a file descriptor if an archive is malformed.

From-SVN: r250684
---

diff --git a/libbacktrace/ChangeLog b/libbacktrace/ChangeLog
index c60f714aa20..313a4c61855 100644
--- a/libbacktrace/ChangeLog
+++ b/libbacktrace/ChangeLog
@@ -1,3 +1,7 @@
+2017-07-28  Tony Reix  <tony.reix@atos.net>
+
+	* xcoff.c: Don't leak a file descriptor if an archive is malformed.
+
 2017-07-28  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>
 
 	* fileline.c (fileline_initialize): Print pid_t as long.
diff --git a/libbacktrace/xcoff.c b/libbacktrace/xcoff.c
index 76210733f01..2faa9fdfa05 100644
--- a/libbacktrace/xcoff.c
+++ b/libbacktrace/xcoff.c
@@ -1288,20 +1288,20 @@ xcoff_armem_add (struct backtrace_state *state, int descriptor,
 
   if (!backtrace_get_view (state, descriptor, 0, sizeof (b_ar_fl_hdr),
 			   error_callback, data, &view))
-    return 0;
+    goto fail;
 
   memcpy (&fl_hdr, view.data, sizeof (b_ar_fl_hdr));
 
   backtrace_release_view (state, &view, error_callback, data);
 
   if (memcmp (fl_hdr.fl_magic, AIAMAGBIG, 8) != 0)
-    return 0;
+    goto fail;
 
   memlen = strlen (member);
 
   /* Read offset of first archive member.  */
   if (!xcoff_parse_decimal (fl_hdr.fl_fstmoff, sizeof fl_hdr.fl_fstmoff, &off))
-    return 0;
+    goto fail;
   while (off != 0)
     {
       /* Map archive member header and member name.  */
@@ -1309,7 +1309,7 @@ xcoff_armem_add (struct backtrace_state *state, int descriptor,
       if (!backtrace_get_view (state, descriptor, off,
 			       sizeof (b_ar_hdr) + memlen,
 			       error_callback, data, &view))
-	return 0;
+	break;
 
       ar_hdr = (const b_ar_hdr *) view.data;
 
@@ -1345,6 +1345,7 @@ xcoff_armem_add (struct backtrace_state *state, int descriptor,
       backtrace_release_view (state, &view, error_callback, data);
     }
 
+ fail:
   /* No matching member found.  */
   backtrace_close (descriptor, error_callback, data);
   return 0;