diff options
author | Dale Rahn <drahn@cvs.openbsd.org> | 2004-05-17 21:54:57 +0000 |
---|---|---|
committer | Dale Rahn <drahn@cvs.openbsd.org> | 2004-05-17 21:54:57 +0000 |
commit | 631eeff208608ed7b44459bd6c8ae905bcd7e8b6 (patch) | |
tree | 7be124105a0929655708fbbc134f9c2e3189c3f3 /gnu/usr.bin/binutils/ld/ldcref.c | |
parent | a7c8e0ae4a4b349099d02bc9d56bcf1979c0f0c3 (diff) |
Resolve merge conflicts, adjust method of W^X handing (.sh files)
remove testsuites (not useable) remove mmalloc (not part of new binutils).
Diffstat (limited to 'gnu/usr.bin/binutils/ld/ldcref.c')
-rw-r--r-- | gnu/usr.bin/binutils/ld/ldcref.c | 199 |
1 files changed, 113 insertions, 86 deletions
diff --git a/gnu/usr.bin/binutils/ld/ldcref.c b/gnu/usr.bin/binutils/ld/ldcref.c index 6b15ffd3ee5..fec77b06413 100644 --- a/gnu/usr.bin/binutils/ld/ldcref.c +++ b/gnu/usr.bin/binutils/ld/ldcref.c @@ -1,5 +1,5 @@ /* ldcref.c -- output a cross reference table - Copyright 1996, 1997, 1998, 2000 Free Software Foundation, Inc. + Copyright 1996, 1997, 1998, 2000, 2002 Free Software Foundation, Inc. Written by Ian Lance Taylor <ian@cygnus.com> This file is part of GLD, the Gnu Linker. @@ -69,13 +69,13 @@ struct cref_hash_table { static struct bfd_hash_entry *cref_hash_newfunc PARAMS ((struct bfd_hash_entry *, struct bfd_hash_table *, const char *)); -static boolean cref_fill_array PARAMS ((struct cref_hash_entry *, PTR)); +static bfd_boolean cref_fill_array PARAMS ((struct cref_hash_entry *, PTR)); static int cref_sort_array PARAMS ((const PTR, const PTR)); static void output_one_cref PARAMS ((FILE *, struct cref_hash_entry *)); -static boolean check_nocrossref PARAMS ((struct cref_hash_entry *, PTR)); +static bfd_boolean check_nocrossref PARAMS ((struct cref_hash_entry *, PTR)); +static void check_section_sym_xref PARAMS ((lang_input_statement_type *)); static void check_refs - PARAMS ((struct cref_hash_entry *, struct bfd_link_hash_entry *, - struct lang_nocrossrefs *)); + PARAMS ((const char *, asection *, bfd *, struct lang_nocrossrefs *)); static void check_reloc_refs PARAMS ((bfd *, asection *, PTR)); /* Look up an entry in the cref hash table. */ @@ -89,7 +89,7 @@ static void check_reloc_refs PARAMS ((bfd *, asection *, PTR)); #define cref_hash_traverse(table, func, info) \ (bfd_hash_traverse \ (&(table)->root, \ - (boolean (*) PARAMS ((struct bfd_hash_entry *, PTR))) (func), \ + (bfd_boolean (*) PARAMS ((struct bfd_hash_entry *, PTR))) (func), \ (info))) /* The cref hash table. */ @@ -98,7 +98,7 @@ static struct cref_hash_table cref_table; /* Whether the cref hash table has been initialized. */ -static boolean cref_initialized; +static bfd_boolean cref_initialized; /* The number of symbols seen so far. */ @@ -156,10 +156,10 @@ add_cref (name, abfd, section, value) { if (! bfd_hash_table_init (&cref_table.root, cref_hash_newfunc)) einfo (_("%X%P: bfd_hash_table_init of cref table failed: %E\n")); - cref_initialized = true; + cref_initialized = TRUE; } - h = cref_hash_lookup (&cref_table, name, true, false); + h = cref_hash_lookup (&cref_table, name, TRUE, FALSE); if (h == NULL) einfo (_("%X%P: cref_hash_lookup failed: %E\n")); @@ -173,24 +173,24 @@ add_cref (name, abfd, section, value) r->next = h->refs; h->refs = r; r->abfd = abfd; - r->def = false; - r->common = false; - r->undef = false; + r->def = FALSE; + r->common = FALSE; + r->undef = FALSE; } if (bfd_is_und_section (section)) - r->undef = true; + r->undef = TRUE; else if (bfd_is_com_section (section)) - r->common = true; + r->common = TRUE; else - r->def = true; + r->def = TRUE; } /* Copy the addresses of the hash table entries into an array. This is called via cref_hash_traverse. We also fill in the demangled name. */ -static boolean +static bfd_boolean cref_fill_array (h, data) struct cref_hash_entry *h; PTR data; @@ -204,7 +204,7 @@ cref_fill_array (h, data) ++*pph; - return true; + return TRUE; } /* Sort an array of cref hash table entries by name. */ @@ -274,8 +274,8 @@ output_one_cref (fp, h) struct bfd_link_hash_entry *hl; struct cref_ref *r; - hl = bfd_link_hash_lookup (link_info.hash, h->root.string, false, - false, true); + hl = bfd_link_hash_lookup (link_info.hash, h->root.string, FALSE, + FALSE, TRUE); if (hl == NULL) einfo ("%P: symbol `%T' missing from main hash table\n", h->root.string); @@ -342,11 +342,46 @@ check_nocrossrefs () return; cref_hash_traverse (&cref_table, check_nocrossref, (PTR) NULL); + + lang_for_each_file (check_section_sym_xref); +} + +/* Checks for prohibited cross references to section symbols. */ + +static void +check_section_sym_xref (statement) + lang_input_statement_type *statement; +{ + bfd *abfd; + asection *sec; + + abfd = statement->the_bfd; + if (abfd == NULL) + return; + + for (sec = abfd->sections; sec != NULL; sec = sec->next) + { + asection *outsec; + + outsec = sec->output_section; + if (outsec != NULL) + { + const char *outsecname; + struct lang_nocrossrefs *ncrs; + struct lang_nocrossref *ncr; + + outsecname = outsec->name; + for (ncrs = nocrossref_list; ncrs != NULL; ncrs = ncrs->next) + for (ncr = ncrs->list; ncr != NULL; ncr = ncr->next) + if (strcmp (ncr->name, outsecname) == 0) + check_refs (NULL, sec, abfd, ncrs); + } + } } /* Check one symbol to see if it is a prohibited cross reference. */ -static boolean +static bfd_boolean check_nocrossref (h, ignore) struct cref_hash_entry *h; PTR ignore ATTRIBUTE_UNUSED; @@ -356,42 +391,43 @@ check_nocrossref (h, ignore) const char *defsecname; struct lang_nocrossrefs *ncrs; struct lang_nocrossref *ncr; + struct cref_ref *ref; - hl = bfd_link_hash_lookup (link_info.hash, h->root.string, false, - false, true); + hl = bfd_link_hash_lookup (link_info.hash, h->root.string, FALSE, + FALSE, TRUE); if (hl == NULL) { einfo (_("%P: symbol `%T' missing from main hash table\n"), h->root.string); - return true; + return TRUE; } if (hl->type != bfd_link_hash_defined && hl->type != bfd_link_hash_defweak) - return true; + return TRUE; defsec = hl->u.def.section->output_section; if (defsec == NULL) - return true; + return TRUE; defsecname = bfd_get_section_name (defsec->owner, defsec); for (ncrs = nocrossref_list; ncrs != NULL; ncrs = ncrs->next) for (ncr = ncrs->list; ncr != NULL; ncr = ncr->next) if (strcmp (ncr->name, defsecname) == 0) - check_refs (h, hl, ncrs); + for (ref = h->refs; ref != NULL; ref = ref->next) + check_refs (hl->root.string, hl->u.def.section, ref->abfd, ncrs); - return true; + return TRUE; } /* The struct is used to pass information from check_refs to check_reloc_refs through bfd_map_over_sections. */ struct check_refs_info { - struct cref_hash_entry *h; + const char *sym_name; asection *defsec; struct lang_nocrossrefs *ncrs; asymbol **asymbols; - boolean same; }; /* This function is called for each symbol defined in a section which @@ -400,70 +436,59 @@ struct check_refs_info { prohibited sections. */ static void -check_refs (h, hl, ncrs) - struct cref_hash_entry *h; - struct bfd_link_hash_entry *hl; +check_refs (name, sec, abfd, ncrs) + const char *name; + asection *sec; + bfd *abfd; struct lang_nocrossrefs *ncrs; { - struct cref_ref *ref; + lang_input_statement_type *li; + asymbol **asymbols; + struct check_refs_info info; - for (ref = h->refs; ref != NULL; ref = ref->next) + /* We need to look through the relocations for this BFD, to see + if any of the relocations which refer to this symbol are from + a prohibited section. Note that we need to do this even for + the BFD in which the symbol is defined, since even a single + BFD might contain a prohibited cross reference. */ + + li = (lang_input_statement_type *) abfd->usrdata; + if (li != NULL && li->asymbols != NULL) + asymbols = li->asymbols; + else { - lang_input_statement_type *li; - asymbol **asymbols; - struct check_refs_info info; - - /* We need to look through the relocations for this BFD, to see - if any of the relocations which refer to this symbol are from - a prohibited section. Note that we need to do this even for - the BFD in which the symbol is defined, since even a single - BFD might contain a prohibited cross reference; for this - case, we set the SAME field in INFO, which will cause - CHECK_RELOCS_REFS to check for relocations against the - section as well as against the symbol. */ - - li = (lang_input_statement_type *) ref->abfd->usrdata; - if (li != NULL && li->asymbols != NULL) - asymbols = li->asymbols; - else + long symsize; + long symbol_count; + + symsize = bfd_get_symtab_upper_bound (abfd); + if (symsize < 0) + einfo (_("%B%F: could not read symbols; %E\n"), abfd); + asymbols = (asymbol **) xmalloc (symsize); + symbol_count = bfd_canonicalize_symtab (abfd, asymbols); + if (symbol_count < 0) + einfo (_("%B%F: could not read symbols: %E\n"), abfd); + if (li != NULL) { - long symsize; - long symbol_count; - - symsize = bfd_get_symtab_upper_bound (ref->abfd); - if (symsize < 0) - einfo (_("%B%F: could not read symbols; %E\n"), ref->abfd); - asymbols = (asymbol **) xmalloc (symsize); - symbol_count = bfd_canonicalize_symtab (ref->abfd, asymbols); - if (symbol_count < 0) - einfo (_("%B%F: could not read symbols: %E\n"), ref->abfd); - if (li != NULL) - { - li->asymbols = asymbols; - li->symbol_count = symbol_count; - } + li->asymbols = asymbols; + li->symbol_count = symbol_count; } - - info.h = h; - info.defsec = hl->u.def.section; - info.ncrs = ncrs; - info.asymbols = asymbols; - if (ref->abfd == hl->u.def.section->owner) - info.same = true; - else - info.same = false; - bfd_map_over_sections (ref->abfd, check_reloc_refs, (PTR) &info); - - if (li == NULL) - free (asymbols); } + + info.sym_name = name; + info.defsec = sec; + info.ncrs = ncrs; + info.asymbols = asymbols; + bfd_map_over_sections (abfd, check_reloc_refs, (PTR) &info); + + if (li == NULL) + free (asymbols); } -/* This is called via bfd_map_over_sections. INFO->H is a symbol +/* This is called via bfd_map_over_sections. INFO->SYM_NAME is a symbol defined in INFO->DEFSECNAME. If this section maps into any of the sections listed in INFO->NCRS, other than INFO->DEFSECNAME, then we look through the relocations. If any of the relocations are to - INFO->H, then we report a prohibited cross reference error. */ + INFO->SYM_NAME, then we report a prohibited cross reference error. */ static void check_reloc_refs (abfd, sec, iarg) @@ -502,9 +527,10 @@ check_reloc_refs (abfd, sec, iarg) /* This section is one for which cross references are prohibited. Look through the relocations, and see if any of them are to - INFO->H. */ + INFO->SYM_NAME. If INFO->SYMNAME is NULL, check for relocations + against the section symbol. */ - symname = info->h->root.string; + symname = info->sym_name; relsize = bfd_get_reloc_upper_bound (abfd, sec); if (relsize < 0) @@ -525,9 +551,10 @@ check_reloc_refs (abfd, sec, iarg) if (q->sym_ptr_ptr != NULL && *q->sym_ptr_ptr != NULL - && (strcmp (bfd_asymbol_name (*q->sym_ptr_ptr), symname) == 0 - || (info->same - && bfd_get_section (*q->sym_ptr_ptr) == info->defsec))) + && (symname != NULL + ? strcmp (bfd_asymbol_name (*q->sym_ptr_ptr), symname) == 0 + : (((*q->sym_ptr_ptr)->flags & BSF_SECTION_SYM) != 0 + && bfd_get_section (*q->sym_ptr_ptr) == info->defsec))) { /* We found a reloc for the symbol. The symbol is defined in OUTSECNAME. This reloc is from a section which is |