aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIan Lance Taylor <ian@airs.com>1994-08-15 22:07:29 +0000
committerIan Lance Taylor <ian@airs.com>1994-08-15 22:07:29 +0000
commit9fbe895ae728fe0eb572d266399e2d86eef5aebd (patch)
tree5bdce7910e1c2e4440c6202d2d547e744f433f0a
parent1fee94ece02eb29ae3c95a87813083907d33274e (diff)
downloadgdb-9fbe895ae728fe0eb572d266399e2d86eef5aebd.zip
gdb-9fbe895ae728fe0eb572d266399e2d86eef5aebd.tar.gz
gdb-9fbe895ae728fe0eb572d266399e2d86eef5aebd.tar.bz2
* coffgen.c (STRING_SIZE_SIZE): Define.
(coff_fix_symbol_name): Use STRING_SIZE_SIZE, not 4. (coff_write_symbols, build_string_table): Likewise. (coff_get_normalized_symtab): Likewise.
-rw-r--r--bfd/ChangeLog5
-rw-r--r--bfd/coffgen.c48
2 files changed, 39 insertions, 14 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index a7d4ead..95ae851 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,5 +1,10 @@
Mon Aug 15 12:16:56 1994 Ian Lance Taylor (ian@sanguine.cygnus.com)
+ * coffgen.c (STRING_SIZE_SIZE): Define.
+ (coff_fix_symbol_name): Use STRING_SIZE_SIZE, not 4.
+ (coff_write_symbols, build_string_table): Likewise.
+ (coff_get_normalized_symtab): Likewise.
+
* libcoff-in.h (obj_symbol_slew): Don't define.
(struct coff_tdata): Remove unused fields index_slew, raw_linenos,
and flags.
diff --git a/bfd/coffgen.c b/bfd/coffgen.c
index 2e1ad2d..556ca3b 100644
--- a/bfd/coffgen.c
+++ b/bfd/coffgen.c
@@ -51,6 +51,8 @@ static boolean coff_write_native_symbol PARAMS ((bfd *, coff_symbol_type *,
static asection bfd_debug_section = { "*DEBUG*" };
+#define STRING_SIZE_SIZE (4)
+
/* Take a section header read from a coff file (in HOST byte order),
and make a BFD "section" out of it. This is used by ECOFF. */
static boolean
@@ -600,7 +602,7 @@ coff_fix_symbol_name (abfd, symbol, native)
strncpy(auxent->x_file.x_fname, name, FILNMLEN);
}
else {
- auxent->x_file.x_n.x_offset = string_size + 4;
+ auxent->x_file.x_n.x_offset = string_size + STRING_SIZE_SIZE;
auxent->x_file.x_n.x_zeroes = 0;
string_size += name_length + 1;
}
@@ -621,7 +623,7 @@ coff_fix_symbol_name (abfd, symbol, native)
}
else if (! bfd_coff_symname_in_debug (abfd, &native->u.syment))
{
- native->u.syment._n._n_n._n_offset = string_size + 4;
+ native->u.syment._n._n_n._n_offset = string_size + STRING_SIZE_SIZE;
native->u.syment._n._n_n._n_zeroes = 0;
string_size += name_length + 1;
}
@@ -918,10 +920,14 @@ coff_write_symbols (abfd)
if (string_size != 0)
{
- unsigned int size = string_size + 4;
- bfd_byte buffer[4];
+ unsigned int size = string_size + STRING_SIZE_SIZE;
+ bfd_byte buffer[STRING_SIZE_SIZE];
+#if STRING_SIZE_SIZE == 4
bfd_h_put_32 (abfd, size, buffer);
+#else
+ #error Change bfd_h_put_32
+#endif
if (bfd_write ((PTR) buffer, 1, sizeof (buffer), abfd) != sizeof (buffer))
return false;
for (p = abfd->outsymbols, i = 0;
@@ -972,11 +978,16 @@ coff_write_symbols (abfd)
/* We would normally not write anything here, but we'll write
out 4 so that any stupid coff reader which tries to read the
string table even when there isn't one won't croak. */
- unsigned int size = 4;
- bfd_byte buffer[4];
+ unsigned int size = STRING_SIZE_SIZE;
+ bfd_byte buffer[STRING_SIZE_SIZE];
+#if STRING_SIZE_SIZE == 4
bfd_h_put_32 (abfd, size, buffer);
- if (bfd_write ((PTR) buffer, 1, 4, abfd) != 4)
+#else
+ #error Change bfd_h_put_32
+#endif
+ if (bfd_write ((PTR) buffer, 1, STRING_SIZE_SIZE, abfd)
+ != STRING_SIZE_SIZE)
return false;
}
@@ -1138,7 +1149,7 @@ static char *
build_string_table (abfd)
bfd *abfd;
{
- char string_table_size_buffer[4];
+ char string_table_size_buffer[STRING_SIZE_SIZE];
unsigned int string_table_size;
char *string_table;
@@ -1149,12 +1160,19 @@ build_string_table (abfd)
1, abfd) != sizeof(string_table_size))
return (NULL);
+#if STRING_SIZE_SIZE == 4
string_table_size = bfd_h_get_32(abfd, (bfd_byte *) string_table_size_buffer);
+#else
+ #error Change bfd_h_get_32
+#endif
- if ((string_table = (PTR) bfd_alloc(abfd, string_table_size -= 4)) == NULL) {
- bfd_set_error (bfd_error_no_memory);
- return (NULL);
- } /* on mallocation error */
+ if ((string_table = (PTR) bfd_alloc(abfd,
+ string_table_size -= STRING_SIZE_SIZE))
+ == NULL)
+ {
+ bfd_set_error (bfd_error_no_memory);
+ return (NULL);
+ } /* on mallocation error */
if (bfd_read(string_table, string_table_size, 1, abfd) != string_table_size)
return (NULL);
return string_table;
@@ -1343,7 +1361,7 @@ coff_get_normalized_symtab (abfd)
}
internal_ptr->u.syment._n._n_n._n_offset =
- (long) (string_table - 4 +
+ (long) (string_table - STRING_SIZE_SIZE +
(internal_ptr+1)->u.auxent.x_file.x_n.x_offset);
}
else {
@@ -1384,7 +1402,9 @@ coff_get_normalized_symtab (abfd)
string_table = build_string_table(abfd);
}
internal_ptr->u.syment._n._n_n._n_offset = (long int)
- (string_table - 4 + internal_ptr->u.syment._n._n_n._n_offset);
+ (string_table
+ - STRING_SIZE_SIZE
+ + internal_ptr->u.syment._n._n_n._n_offset);
}
else {
/* Long name in debug section. Very similar. */