On Wed, Jan 11, 2006 at 08:43:53AM +1030, Alan Modra wrote:
> * objcopy.c (copy_object): Set isympp and osympp to NULL after free.
Reviewing my own patch before committing would have been better..
There was a good reason why the test is "osympp != isympp" rather than
"osympp != NULL", and changing isympp too early breaks this test.
* objcopy.c (copy_object): Fix thinko.
Index: binutils/objcopy.c
===================================================================
RCS file: /cvs/src/src/binutils/objcopy.c,v
retrieving revision 1.91
diff -u -p -r1.91 objcopy.c
--- binutils/objcopy.c 10 Jan 2006 22:14:18 -0000 1.91
+++ binutils/objcopy.c 10 Jan 2006 22:24:01 -0000
@@ -1281,16 +1281,13 @@ copy_object (bfd *ibfd, bfd *obfd)
}
if (isympp)
- {
- free (isympp);
- isympp = NULL;
- }
+ free (isympp);
if (osympp != isympp)
- {
- free (osympp);
- osympp = NULL;
- }
+ free (osympp);
+
+ isympp = NULL;
+ osympp = NULL;
/* BFD mandates that all output sections be created and sizes set before
any output is done. Thus, we traverse all sections multiple times. */
--
Alan Modra
IBM OzLabs - Linux Technology Centre
_______________________________________________
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils
