Perform cleanup in bfd_close after errors
authorAlan Modra <amodra@gmail.com>
Fri, 27 Jan 2023 04:46:03 +0000 (15:16 +1030)
committerAlan Modra <amodra@gmail.com>
Fri, 27 Jan 2023 05:16:45 +0000 (15:46 +1030)
It seems reasonable to continue after errors in bfd_close_all_done,
particularly since bfd_close_all_done is typically called on an output
file after we've hit some sort of error elsewhere.  The iovec test is
necessary if bfd_close_all_done is to work on odd bfd's opened by
bfd_create.

* opncls.c (bfd_close): Call bfd_close_all_done after errors
from _bfd_write_contents.
(bfd_close_all_done): Call _bfd_delete_bfd after errors.
Don't call iovec->bclose when iovec is NULL.

bfd/opncls.c

index 9241cd1c537bad7594b843bee2c4fa7680c7734f..6ae3af054e4b6d9457e3dc1cd0659db4a8247811 100644 (file)
@@ -805,13 +805,10 @@ RETURNS
 bool
 bfd_close (bfd *abfd)
 {
-  if (bfd_write_p (abfd))
-    {
-      if (! BFD_SEND_FMT (abfd, _bfd_write_contents, (abfd)))
-       return false;
-    }
+  bool ret = (!bfd_write_p (abfd)
+             || BFD_SEND_FMT (abfd, _bfd_write_contents, (abfd)));
 
-  return bfd_close_all_done (abfd);
+  return bfd_close_all_done (abfd) && ret;
 }
 
 /*
@@ -839,15 +836,15 @@ RETURNS
 bool
 bfd_close_all_done (bfd *abfd)
 {
-  bool ret;
+  bool ret = BFD_SEND (abfd, _close_and_cleanup, (abfd));
 
-  if (! BFD_SEND (abfd, _close_and_cleanup, (abfd)))
-    return false;
-
-  ret = abfd->iovec->bclose (abfd) == 0;
+  if (ret && abfd->iovec != NULL)
+    {
+      ret = abfd->iovec->bclose (abfd) == 0;
 
-  if (ret)
-    _maybe_make_executable (abfd);
+      if (ret)
+       _maybe_make_executable (abfd);
+    }
 
   _bfd_delete_bfd (abfd);