[PATCH] tests: Fix cfi_debug_bias assert in varlocs.

Mark Wielaard mark@klomp.org
Fri Jun 8 14:07:00 GMT 2018


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 <mark@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  <mark@klomp.org>
+
+	* varlocs.c (main): Only assert when cfi_debug_bias != 0 if there
+	actually is a cfi_debug.
+
 2018-06-07  Mark Wielaard  <mark@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



More information about the Elfutils-devel mailing list