It is only a consistency issue if we actually have an cfi_debug and the cfi_debug_bias is not zero (because they come from the same file as the other debug data).
Signed-off-by: Mark Wielaard <m...@klomp.org> --- tests/ChangeLog | 5 +++++ tests/varlocs.c | 3 ++- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/tests/ChangeLog b/tests/ChangeLog index a2adfee..25ed41e 100644 --- a/tests/ChangeLog +++ b/tests/ChangeLog @@ -1,3 +1,8 @@ +2018-06-08 Mark Wielaard <m...@klomp.org> + + * varlocs.c (main): Only assert when cfi_debug_bias != 0 if there + actually is a cfi_debug. + 2018-06-07 Mark Wielaard <m...@klomp.org> * run-readelf-loc.sh: Fix expected output for startx_length. diff --git a/tests/varlocs.c b/tests/varlocs.c index 2ddd3d8..99c3887 100644 --- a/tests/varlocs.c +++ b/tests/varlocs.c @@ -1121,7 +1121,8 @@ main (int argc, char *argv[]) cfi_debug = dwfl_module_dwarf_cfi (mod, &cfi_debug_bias); cfi_eh = dwfl_module_eh_cfi (mod, &cfi_eh_bias); - assert (cfi_debug_bias == 0); // No bias needed, same file. + // No bias needed, same file. + assert (cfi_debug == NULL || cfi_debug_bias == 0); // We are a bit forgiving for object files. There might be // relocations we don't handle that are needed in some -- 1.8.3.1