aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--bfd/ChangeLog20
-rw-r--r--bfd/coff-stgo32.c96
-rw-r--r--bfd/coffcode.h5
-rw-r--r--bfd/libcoff-in.h3
-rw-r--r--bfd/libcoff.h3
-rw-r--r--include/coff/ChangeLog9
-rw-r--r--include/coff/go32exe.h6
-rw-r--r--include/coff/internal.h13
8 files changed, 98 insertions, 57 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index d34a4f2..a112ee3 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,5 +1,25 @@
2009-08-10 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.
+
+2009-08-10 Jan Kratochvil <jan.kratochvil@redhat.com>
+
Fix go32 stub preservation by objcopy.
* coff-stgo32.c (adjust_filehdr_in_post): Use bfd_malloc.
(go32_stubbed_coff_bfd_copy_private_bfd_data): Optionally allocate OBFD
diff --git a/bfd/coff-stgo32.c b/bfd/coff-stgo32.c
index 51ea732e..9b43cb6 100644
--- a/bfd/coff-stgo32.c
+++ b/bfd/coff-stgo32.c
@@ -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[STUBSIZE] =
/*
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[STUBSIZE] =
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);
-
- /* 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) = bfd_malloc ((bfd_size_type) STUBSIZE);
+ ADJUST_VAL (filehdr_dst->f_symptr, GO32_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, class, indx, numaux, in1)
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, class, indx, numaux, extp)
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, class, indx, numaux, extp)
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,31 @@ create_go32_stub (abfd)
goto stub_end;
}
/* Now we found a correct stub (hopefully). */
- bfd_coff_go32stub (abfd)
- = (PTR) bfd_alloc (abfd, (bfd_size_type) coff_start);
- if (bfd_coff_go32stub (abfd) == NULL)
+ coff_data (abfd)->go32stub = bfd_alloc (abfd, (bfd_size_type) coff_start);
+ 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
+ = 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 +395,19 @@ go32_stubbed_coff_bfd_copy_private_bfd_data (ibfd, obfd)
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) = bfd_alloc (obfd, (bfd_size_type) STUBSIZE);
+ if (coff_data (obfd)->go32stub == NULL)
+ coff_data (obfd)->go32stub = 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;
}
diff --git a/bfd/coffcode.h b/bfd/coffcode.h
index 4b748b4..1cc860a 100644
--- a/bfd/coffcode.h
+++ b/bfd/coffcode.h
@@ -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
diff --git a/bfd/libcoff-in.h b/bfd/libcoff-in.h
index 16b6c7b..f3108ee 100644
--- a/bfd/libcoff-in.h
+++ b/bfd/libcoff-in.h
@@ -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. */
diff --git a/bfd/libcoff.h b/bfd/libcoff.h
index a4ab405..4bb43f3 100644
--- a/bfd/libcoff.h
+++ b/bfd/libcoff.h
@@ -110,6 +110,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. */
diff --git a/include/coff/ChangeLog b/include/coff/ChangeLog
index db6c55a..a38228b 100644
--- a/include/coff/ChangeLog
+++ b/include/coff/ChangeLog
@@ -1,3 +1,12 @@
+2009-08-10 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.
+
2009-06-03 Ulrich Weigand <uweigand@de.ibm.com>
* symconst.h (btLong64, btULong64, btLongLong64, btULongLong64,
diff --git a/include/coff/go32exe.h b/include/coff/go32exe.h
index c85da0c..2248ff2 100644
--- a/include/coff/go32exe.h
+++ b/include/coff/go32exe.h
@@ -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
diff --git a/include/coff/internal.h b/include/coff/internal.h
index c5d6cd1..36d03eb 100644
--- a/include/coff/internal.h
+++ b/include/coff/internal.h
@@ -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