From 84b2cb0e20adc129e915644b55aba7d93243b2cc Mon Sep 17 00:00:00 2001 From: patacongo Date: Tue, 23 Jun 2009 18:55:57 +0000 Subject: Remove import symbols from relo table git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@1930 42af7a65-404d-4744-a932-0658087f49c3 --- nuttx/binfmt/libnxflat/libnxflat_bind.c | 28 ++++++++-------------------- 1 file changed, 8 insertions(+), 20 deletions(-) (limited to 'nuttx/binfmt/libnxflat/libnxflat_bind.c') diff --git a/nuttx/binfmt/libnxflat/libnxflat_bind.c b/nuttx/binfmt/libnxflat/libnxflat_bind.c index 9ed623397..98b00996b 100644 --- a/nuttx/binfmt/libnxflat/libnxflat_bind.c +++ b/nuttx/binfmt/libnxflat/libnxflat_bind.c @@ -186,9 +186,7 @@ static inline int nxflat_gotrelocs(FAR struct nxflat_loadinfo_s *loadinfo) DEBUGASSERT(offset >= loadinfo->isize && offset < (loadinfo->isize + loadinfo->dsize)); relocs = (FAR struct nxflat_reloc_s*)(offset - loadinfo->isize + loadinfo->dspace->region); - /* Now, traverse the relocation list of imported symbols and attempt to bind - * each GOT relocation (imported symbols will be handled elsewhere). - */ + /* Now, traverse the relocation list of and bind each GOT relocation. */ ret = OK; /* Assume success */ for (i = 0; i < nrelocs; i++) @@ -223,16 +221,10 @@ static inline int nxflat_gotrelocs(FAR struct nxflat_loadinfo_s *loadinfo) } break; - /* NXFLAT_RELOC_TYPE_ABS32 Meaning: Offset refers to a struct nxflat_import_s - * describing a function pointer to be - * imported. - * Fixup: Provide the absolute function address - * in the struct nxflat_import_s instance. - */ - - case NXFLAT_RELOC_TYPE_ABS32: + default: { - /* These will be handled together in nxflat_bindimports */ + bdbg("ERROR: Unrecognized relocation type: %d\n", NXFLAT_RELOC_TYPE(reloc.r_info)); + result = -EINVAL; } break; } @@ -370,22 +362,18 @@ static inline int nxflat_bindimports(FAR struct nxflat_loadinfo_s *loadinfo, int nxflat_bind(FAR struct nxflat_loadinfo_s *loadinfo, FAR const struct symtab_s *exports, int nexports) { - /* First bind all GOT relocations (omitting absolute symbol relocations) */ + /* First bind all GOT relocations */ int ret = nxflat_gotrelocs(loadinfo); if (ret == OK) { - /* Then bind the imported symbol, absolute relocations separately. - * There is no particular reason to do these separately over than - * traversing the import list directly is simpler than traversing - * it indirectly through the relocation list. - */ + /* Then bind the imported symbol, absolute relocations separately. */ ret = nxflat_bindimports(loadinfo, exports, nexports); if (ret == OK) { - /* Zero the BSS area, trashing the relocations that lived in space - * in the file. + /* Zero the BSS area, trashing the relocations that lived in that + * space in the loaded file. */ memset((void*)(loadinfo->dspace->region + loadinfo->datasize), -- cgit v1.2.3