[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