RepoMirrors
/
kpatch
mirror of https://github.com/dynup/kpatch
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

lookup: add duplicate symbol checks 

Add checks for duplicate symbols, and refactor the logic slightly.

Signed-off-by: Josh Poimboeuf <jpoimboe@redhat.com>
This commit is contained in:
Josh Poimboeuf 2019-11-01 15:19:55 -05:00
parent 3064cf3c60
commit ae9f1c17b1
1 changed files with 19 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
kpatch-build/lookup.c
View File

@ -404,7 +404,7 @@ bool lookup_local_symbol(struct lookup_table *table, char *name,
{
struct object_symbol *sym;
unsigned long sympos = 0;
int i, match = 0, in_file = 0;
int i, in_file = 0;
if (!table->local_syms)
return false;
@ -426,19 +426,18 @@ bool lookup_local_symbol(struct lookup_table *table, char *name,
break;
if (sym->bind == STB_LOCAL && !strcmp(sym->name, name)) {
match = 1;
break;
}
}
if (!match)
return false;
if (result->objname)
ERROR("duplicate local symbol found for %s", name);
result->objname = table->objname;
result->sympos = sympos;
result->addr = sym->addr;
result->size = sym->size;
return true;
result->sympos = sympos;
}
}
return !!result->objname;
}
bool lookup_global_symbol(struct lookup_table *table, char *name,
@ -451,15 +450,18 @@ bool lookup_global_symbol(struct lookup_table *table, char *name,
for_each_obj_symbol(i, sym, table) {
if ((sym->bind == STB_GLOBAL || sym->bind == STB_WEAK) &&
!strcmp(sym->name, name)) {
if (result->objname)
ERROR("duplicate global symbol found for %s", name);
result->objname = table->objname;
result->addr = sym->addr;
result->size = sym->size;
result->sympos = 0; /* always 0 for global symbols */
return true;
}
}
return false;
return !!result->objname;
}
bool lookup_is_exported_symbol(struct lookup_table *table, char *name)