Messages in this thread Patch in this message | | | From | Kristen Carlson Accardi <> | Subject | [PATCH v4 01/10] objtool: Do not assume order of parent/child functions | Date | Fri, 17 Jul 2020 09:59:58 -0700 |
| |
If a .cold function is examined prior to it's parent, the link to the parent/child function can be overwritten when the parent is examined. Only update pfunc and cfunc if they were previously nil to prevent this from happening.
Signed-off-by: Kristen Carlson Accardi <kristen@linux.intel.com> Acked-by: Josh Poimboeuf <jpoimboe@redhat.com> --- tools/objtool/elf.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/tools/objtool/elf.c b/tools/objtool/elf.c index 26d11d821941..c18f0f216740 100644 --- a/tools/objtool/elf.c +++ b/tools/objtool/elf.c @@ -434,7 +434,13 @@ static int read_symbols(struct elf *elf) size_t pnamelen; if (sym->type != STT_FUNC) continue; - sym->pfunc = sym->cfunc = sym; + + if (sym->pfunc == NULL) + sym->pfunc = sym; + + if (sym->cfunc == NULL) + sym->cfunc = sym; + coldstr = strstr(sym->name, ".cold"); if (!coldstr) continue; -- 2.20.1
| |