This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
[patch 3/4] Stop using bfd_usrdata in libbfd
- From: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- To: binutils at sourceware dot org
- Date: Sun, 9 Aug 2009 23:40:15 +0200
- Subject: [patch 3/4] Stop using bfd_usrdata in libbfd
Hi,
this is the primary patch to stop using bfd_usrdata in libbfd.
Thanks,
Jan
bfd/
2009-08-09 Jan Kratochvil <jan.kratochvil@redhat.com>
Stop using bfd_usrdata in libbfd.
* coff-stgo32.c (bfd_coff_go32stub): Remove.
(stub_bytes, comment): Replace STUBSIZE by GO32_STUBSIZE.
(adjust_filehdr_in_post): Declare the abfd parameter as unused.
Replace STUBSIZE by GO32_STUBSIZE. Save now the stub in
filehdr_dst->u.go32.stub. New comment with the reason.
(adjust_filehdr_out_pre): Replace STUBSIZE by GO32_STUBSIZE.
Substitute the removed macro bfd_coff_go32stub.
(adjust_filehdr_out_post, adjust_scnhdr_in_post, adjust_scnhdr_out_pre)
(adjust_scnhdr_out_post, adjust_aux_in_post, adjust_aux_out_pre)
(adjust_aux_out_post): Replace STUBSIZE by GO32_STUBSIZE.
(create_go32_stub, go32_stubbed_coff_bfd_copy_private_bfd_data):
Replace STUBSIZE by GO32_STUBSIZE. Substitute the removed macro
bfd_coff_go32stub.
* coffcode.h (coff_mkobject_hook): Initialize coff->go32stub.
* libcoff-in.h (coff_data_type): New field go32stub.
* libcoff.h: Regenerated.
include/coff/
2009-08-09 Jan Kratochvil <jan.kratochvil@redhat.com>
Stop using bfd_usrdata in libbfd.
* go32exe.h (struct external_filehdr_go32_exe <stub>, FILHSZ): Replace
STUBSIZE by GO32_STUBSIZE.
(STUBSIZE): Move the definition ...
* internal.h (GO32_STUBSIZE): ... here and rename it.
(struct internal_filehdr <go32stub>, F_GO32STUB): New.
--- bfd/coff-stgo32.c 2009-08-09 20:31:45.000000000 +0200
+++ bfd/coff-stgo32.c 2009-08-09 20:39:39.000000000 +0200
@@ -102,9 +102,6 @@ static bfd_boolean
#include "coff-i386.c"
-/* I hold in the usrdata the stub. */
-#define bfd_coff_go32stub bfd_usrdata
-
/* This macro is used, because I cannot assume the endianess of the
host system. */
#define _H(index) (H_GET_16 (abfd, (header+index*2)))
@@ -112,7 +109,7 @@ static bfd_boolean
/* These bytes are a 2048-byte DOS executable, which loads the COFF
image into memory and then runs it. It is called 'stub'. */
-static const unsigned char stub_bytes[STUBSIZE] =
+static const unsigned char stub_bytes[GO32_STUBSIZE] =
{
#include "go32stub.h"
};
@@ -120,7 +117,7 @@ static const unsigned char stub_bytes[ST
/*
I have not commented each swap function below, because the
technique is in any function the same. For the ...in function,
- all the pointers are adjusted by adding STUBSIZE and for the
+ all the pointers are adjusted by adding GO32_STUBSIZE and for the
...out function, it is subtracted first and after calling the
standard swap function it is reset to the old value. */
@@ -132,26 +129,20 @@ static const unsigned char stub_bytes[ST
static void
adjust_filehdr_in_post (abfd, src, dst)
- bfd *abfd;
+ bfd *abfd ATTRIBUTE_UNUSED;
PTR src;
PTR dst;
{
FILHDR *filehdr_src = (FILHDR *) src;
struct internal_filehdr *filehdr_dst = (struct internal_filehdr *) dst;
- ADJUST_VAL (filehdr_dst->f_symptr, STUBSIZE);
+ ADJUST_VAL (filehdr_dst->f_symptr, GO32_STUBSIZE);
- /* Save now the stub to be used later. FIXME: Memory leak as the caller
- coff_object_p does bfd_release afterwards. */
- bfd_coff_go32stub (abfd) = (PTR) bfd_malloc ((bfd_size_type) STUBSIZE);
-
- /* Since this function returns no status, I do not set here
- any bfd_error_...
- That means, before the use of bfd_coff_go32stub (), this value
- should be checked if it is != NULL. */
- if (bfd_coff_go32stub (abfd) == NULL)
- return;
- memcpy (bfd_coff_go32stub (abfd), filehdr_src->stub, STUBSIZE);
+ /* Save now the stub to be used later. Put the stub data to FILEHDR_DST
+ first as coff_data (abfd) still does not exist. It may not even be ever
+ created as we are just checking the file format of ABFD. */
+ memcpy (filehdr_dst->go32stub, filehdr_src->stub, GO32_STUBSIZE);
+ filehdr_dst->f_flags |= F_GO32STUB;
}
static void
@@ -167,13 +158,13 @@ adjust_filehdr_out_pre (abfd, in, out)
create_go32_stub (abfd);
/* Copy the stub to the file header. */
- if (bfd_coff_go32stub (abfd) != NULL)
- memcpy (filehdr_out->stub, bfd_coff_go32stub (abfd), STUBSIZE);
+ if (coff_data (abfd)->go32stub != NULL)
+ memcpy (filehdr_out->stub, coff_data (abfd)->go32stub, GO32_STUBSIZE);
else
/* Use the default. */
- memcpy (filehdr_out->stub, stub_bytes, STUBSIZE);
+ memcpy (filehdr_out->stub, stub_bytes, GO32_STUBSIZE);
- ADJUST_VAL (filehdr_in->f_symptr, -STUBSIZE);
+ ADJUST_VAL (filehdr_in->f_symptr, -GO32_STUBSIZE);
}
static void
@@ -184,7 +175,7 @@ adjust_filehdr_out_post (abfd, in, out)
{
struct internal_filehdr *filehdr_in = (struct internal_filehdr *) in;
/* Undo the above change. */
- ADJUST_VAL (filehdr_in->f_symptr, STUBSIZE);
+ ADJUST_VAL (filehdr_in->f_symptr, GO32_STUBSIZE);
}
static void
@@ -195,9 +186,9 @@ adjust_scnhdr_in_post (abfd, ext, in)
{
struct internal_scnhdr *scnhdr_int = (struct internal_scnhdr *) in;
- ADJUST_VAL (scnhdr_int->s_scnptr, STUBSIZE);
- ADJUST_VAL (scnhdr_int->s_relptr, STUBSIZE);
- ADJUST_VAL (scnhdr_int->s_lnnoptr, STUBSIZE);
+ ADJUST_VAL (scnhdr_int->s_scnptr, GO32_STUBSIZE);
+ ADJUST_VAL (scnhdr_int->s_relptr, GO32_STUBSIZE);
+ ADJUST_VAL (scnhdr_int->s_lnnoptr, GO32_STUBSIZE);
}
static void
@@ -208,9 +199,9 @@ adjust_scnhdr_out_pre (abfd, in, out)
{
struct internal_scnhdr *scnhdr_int = (struct internal_scnhdr *) in;
- ADJUST_VAL (scnhdr_int->s_scnptr, -STUBSIZE);
- ADJUST_VAL (scnhdr_int->s_relptr, -STUBSIZE);
- ADJUST_VAL (scnhdr_int->s_lnnoptr, -STUBSIZE);
+ ADJUST_VAL (scnhdr_int->s_scnptr, -GO32_STUBSIZE);
+ ADJUST_VAL (scnhdr_int->s_relptr, -GO32_STUBSIZE);
+ ADJUST_VAL (scnhdr_int->s_lnnoptr, -GO32_STUBSIZE);
}
static void
@@ -221,9 +212,9 @@ adjust_scnhdr_out_post (abfd, in, out)
{
struct internal_scnhdr *scnhdr_int = (struct internal_scnhdr *) in;
- ADJUST_VAL (scnhdr_int->s_scnptr, STUBSIZE);
- ADJUST_VAL (scnhdr_int->s_relptr, STUBSIZE);
- ADJUST_VAL (scnhdr_int->s_lnnoptr, STUBSIZE);
+ ADJUST_VAL (scnhdr_int->s_scnptr, GO32_STUBSIZE);
+ ADJUST_VAL (scnhdr_int->s_relptr, GO32_STUBSIZE);
+ ADJUST_VAL (scnhdr_int->s_lnnoptr, GO32_STUBSIZE);
}
static void
@@ -240,7 +231,7 @@ adjust_aux_in_post (abfd, ext1, type, c
if (class == C_BLOCK || class == C_FCN || ISFCN (type) || ISTAG (class))
{
- ADJUST_VAL (in->x_sym.x_fcnary.x_fcn.x_lnnoptr, STUBSIZE);
+ ADJUST_VAL (in->x_sym.x_fcnary.x_fcn.x_lnnoptr, GO32_STUBSIZE);
}
}
@@ -258,7 +249,7 @@ adjust_aux_out_pre (abfd, inp, type, cl
if (class == C_BLOCK || class == C_FCN || ISFCN (type) || ISTAG (class))
{
- ADJUST_VAL (in->x_sym.x_fcnary.x_fcn.x_lnnoptr, -STUBSIZE);
+ ADJUST_VAL (in->x_sym.x_fcnary.x_fcn.x_lnnoptr, -GO32_STUBSIZE);
}
}
@@ -276,7 +267,7 @@ adjust_aux_out_post (abfd, inp, type, cl
if (class == C_BLOCK || class == C_FCN || ISFCN (type) || ISTAG (class))
{
- ADJUST_VAL (in->x_sym.x_fcnary.x_fcn.x_lnnoptr, STUBSIZE);
+ ADJUST_VAL (in->x_sym.x_fcnary.x_fcn.x_lnnoptr, GO32_STUBSIZE);
}
}
@@ -297,7 +288,7 @@ create_go32_stub (abfd)
bfd *abfd;
{
/* Do it only once. */
- if (bfd_coff_go32stub (abfd) == NULL)
+ if (coff_data (abfd)->go32stub == NULL)
{
char *stub;
struct stat st;
@@ -363,32 +354,32 @@ create_go32_stub (abfd)
goto stub_end;
}
/* Now we found a correct stub (hopefully). */
- bfd_coff_go32stub (abfd)
+ coff_data (abfd)->go32stub
= (PTR) bfd_alloc (abfd, (bfd_size_type) coff_start);
- if (bfd_coff_go32stub (abfd) == NULL)
+ if (coff_data (abfd)->go32stub == NULL)
{
close (f);
return;
}
lseek (f, 0L, SEEK_SET);
- if ((unsigned long) read (f, bfd_coff_go32stub (abfd), coff_start)
+ if ((unsigned long) read (f, coff_data (abfd)->go32stub, coff_start)
!= coff_start)
{
- bfd_release (abfd, bfd_coff_go32stub (abfd));
- bfd_coff_go32stub (abfd) = NULL;
+ bfd_release (abfd, coff_data (abfd)->go32stub);
+ coff_data (abfd)->go32stub = NULL;
}
close (f);
}
stub_end:
/* There was something wrong above, so use now the standard builtin
stub. */
- if (bfd_coff_go32stub (abfd) == NULL)
+ if (coff_data (abfd)->go32stub == NULL)
{
- bfd_coff_go32stub (abfd)
- = (PTR) bfd_alloc (abfd, (bfd_size_type) STUBSIZE);
- if (bfd_coff_go32stub (abfd) == NULL)
+ coff_data (abfd)->go32stub
+ = (PTR) bfd_alloc (abfd, (bfd_size_type) GO32_STUBSIZE);
+ if (coff_data (abfd)->go32stub == NULL)
return;
- memcpy (bfd_coff_go32stub (abfd), stub_bytes, STUBSIZE);
+ memcpy (coff_data (abfd)->go32stub, stub_bytes, GO32_STUBSIZE);
}
}
@@ -405,17 +396,19 @@ go32_stubbed_coff_bfd_copy_private_bfd_d
return TRUE;
/* Check if we have a source stub. */
- if (bfd_coff_go32stub (ibfd) == NULL)
+ if (coff_data (ibfd)->go32stub == NULL)
return TRUE;
/* As adjust_filehdr_out_pre may get called only after this function,
optionally allocate the output stub. */
- if (bfd_coff_go32stub (obfd) == NULL)
- bfd_coff_go32stub (obfd) = (PTR) bfd_alloc (obfd, (bfd_size_type) STUBSIZE);
+ if (coff_data (obfd)->go32stub == NULL)
+ coff_data (obfd)->go32stub = (PTR) bfd_alloc (obfd,
+ (bfd_size_type) GO32_STUBSIZE);
/* Now copy the stub. */
- if (bfd_coff_go32stub (obfd) != NULL)
- memcpy (bfd_coff_go32stub (obfd), bfd_coff_go32stub (ibfd), STUBSIZE);
+ if (coff_data (obfd)->go32stub != NULL)
+ memcpy (coff_data (obfd)->go32stub, coff_data (ibfd)->go32stub,
+ GO32_STUBSIZE);
return TRUE;
}
--- bfd/coffcode.h 2009-07-03 18:07:38.000000000 +0200
+++ bfd/coffcode.h 2009-08-09 20:39:39.000000000 +0200
@@ -1992,6 +1992,11 @@ coff_mkobject_hook (bfd * abfd,
abfd->flags |= HAS_DEBUG;
#endif
+ if ((internal_f->f_flags & F_GO32STUB) != 0)
+ coff->go32stub = (char *) bfd_alloc (abfd, (bfd_size_type) GO32_STUBSIZE);
+ if (coff->go32stub != NULL)
+ memcpy (coff->go32stub, internal_f->go32stub, GO32_STUBSIZE);
+
return coff;
}
#endif
--- bfd/libcoff-in.h 2009-04-21 19:08:20.000000000 +0200
+++ bfd/libcoff-in.h 2009-08-09 20:39:40.000000000 +0200
@@ -106,6 +106,9 @@ typedef struct coff_tdata
used by ARM code. */
flagword flags;
+ /* coff-stgo32 EXE stub header after BFD tdata has been allocated. Its data
+ is kept in internal_filehdr.go32stub beforehand. */
+ char *go32stub;
} coff_data_type;
/* Tdata for pe image files. */
--- include/coff/go32exe.h 2005-05-10 12:21:09.000000000 +0200
+++ include/coff/go32exe.h 2009-08-09 20:39:40.000000000 +0200
@@ -16,11 +16,9 @@
along with this program; if not, write to the Free Software
Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston, MA 02110-1301, USA. */
-#define STUBSIZE 2048
-
struct external_filehdr_go32_exe
{
- char stub[STUBSIZE];/* the stub to load the image */
+ char stub[GO32_STUBSIZE]; /* the stub to load the image */
/* the standard COFF header */
char f_magic[2]; /* magic number */
char f_nscns[2]; /* number of sections */
@@ -34,4 +32,4 @@ struct external_filehdr_go32_exe
#undef FILHDR
#define FILHDR struct external_filehdr_go32_exe
#undef FILHSZ
-#define FILHSZ STUBSIZE+20
+#define FILHSZ GO32_STUBSIZE+20
--- include/coff/internal.h 2009-03-14 10:34:26.000000000 +0100
+++ include/coff/internal.h 2009-08-09 20:39:40.000000000 +0200
@@ -58,10 +58,19 @@ struct internal_extra_pe_filehdr
bfd_vma nt_signature; /* required NT signature, 0x4550 */
};
+#define GO32_STUBSIZE 2048
+
struct internal_filehdr
{
struct internal_extra_pe_filehdr pe;
+ /* coff-stgo32 EXE stub header before BFD tdata has been allocated.
+ Its data is kept in INTERNAL_FILEHDR.GO32STUB afterwards.
+
+ F_GO32STUB is set iff go32stub contains a valid data. Artifical headers
+ created in BFD have no pre-set go32stub. */
+ char go32stub[GO32_STUBSIZE];
+
/* Standard coff internal info. */
unsigned short f_magic; /* magic number */
unsigned short f_nscns; /* number of sections */
@@ -84,7 +93,8 @@ struct internal_filehdr
F_AR32W file is 32-bit big-endian
F_DYNLOAD rs/6000 aix: dynamically loadable w/imports & exports
F_SHROBJ rs/6000 aix: file is a shared object
- F_DLL PE format DLL. */
+ F_DLL PE format DLL
+ F_GO32STUB Field go32stub contains valid data. */
#define F_RELFLG (0x0001)
#define F_EXEC (0x0002)
@@ -96,6 +106,7 @@ struct internal_filehdr
#define F_DYNLOAD (0x1000)
#define F_SHROBJ (0x2000)
#define F_DLL (0x2000)
+#define F_GO32STUB (0x4000)
/* Extra structure which is used in the optional header. */
typedef struct _IMAGE_DATA_DIRECTORY