Re: [RFC/RFA] dangling bfd pointer in archive cache...

[Joel Brobecker <brobecker@adacore.com>]

[-- Attachment #1: Type: text/plain, Size: 366 bytes --]

> On Mon, Oct 15, 2012 at 11:00:52AM -0700, Joel Brobecker wrote:
> >         * coff-rs6000.c (rs6000coff_vec): Set _close_and_cleanup
> >         field to _bfd_archive_close_and_cleanup.
> >         (pmac_xcoff_vec): Likewise.
> 
> OK.  Please apply the same fix to coff64-rs6000.c.

Thanks. Checked in (attached is the patch applied to coff64-rs6000.c).

-- 
Joel

[-- Attachment #2: 0001-dangling-pointer-in-coff64-rs6000-archive-cache.patch --]
[-- Type: text/x-diff, Size: 1070 bytes --]

From ecaaf1440801831e107c39fea3016e2e57cc507c Mon Sep 17 00:00:00 2001
From: Joel Brobecker <brobecker@adacore.com>
Date: Tue, 16 Oct 2012 15:51:13 -0700
Subject: [PATCH] dangling pointer in coff64-rs6000 archive cache.

bfd/ChangeLog:

        * coff64-rs6000.c (rs6000coff64_vec): Set _close_and_cleanup
        field to _bfd_archive_close_and_cleanup.
        (aix5coff64_vec): Likewise.
---
 bfd/coff64-rs6000.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/bfd/coff64-rs6000.c b/bfd/coff64-rs6000.c
index 0821b6f..5f4a502 100644
--- a/bfd/coff64-rs6000.c
+++ b/bfd/coff64-rs6000.c
@@ -2668,7 +2668,7 @@ const bfd_target rs6000coff64_vec =
     },
 
     /* Generic */
-    bfd_true,
+    _bfd_archive_close_and_cleanup,
     bfd_true,
     coff_new_section_hook,
     _bfd_generic_get_section_contents,
@@ -2926,7 +2926,7 @@ const bfd_target aix5coff64_vec =
     },
 
     /* Generic */
-    bfd_true,
+    _bfd_archive_close_and_cleanup,
     bfd_true,
     coff_new_section_hook,
     _bfd_generic_get_section_contents,
-- 
1.7.9.5