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

Reply via email to