A STT_SECTION symbol is not needed if if it is not used as a relocation target. Therefore, a section, in this case a debug section, may not have a secsym associated with it.
Signed-off-by: Bill Wendling <mo...@google.com> Origin: https://github.com/dynup/kpatch.git ba3defa06073 Signed-off-by: Sarah Newman <sarah.new...@computer.org> Reviewed-by: Ross Lagerwall <ross.lagerw...@citrix.com> --- Changes in v3: - add reviewed-by given to v1 of this patch - restored tag from original commit per sending-patches.pandoc Changes in v2: - commit message changed to use Origin --- create-diff-object.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/create-diff-object.c b/create-diff-object.c index a516670..780e6c8 100644 --- a/create-diff-object.c +++ b/create-diff-object.c @@ -1484,7 +1484,7 @@ static void kpatch_include_debug_sections(struct kpatch_elf *kelf) list_for_each_entry(sec, &kelf->sections, list) { if (is_debug_section(sec)) { sec->include = 1; - if (!is_rela_section(sec)) + if (!is_rela_section(sec) && sec->secsym) sec->secsym->include = 1; } } -- 2.17.1
