From: Mark Wielaard <mjw@redhat.com>
Date: Tue, 5 Nov 2013 10:27:19 +0000 (+0100)
Subject: tests: allfcts.c (main): Correct dwarf_getfuncs return value check.
X-Git-Tag: elfutils-0.158~82
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=d8c75df969fff193d143789cba03038db8da5a2c;p=thirdparty%2Felfutils.git

tests: allfcts.c (main): Correct dwarf_getfuncs return value check.

The return value of dwarf_getfuncs is a ptrdiff_t that is zero on success,
or non-zero (an offset to continue the search) when the callback returned
DWARF_CB_ABORT or on error. When an error occurs dwarf_errno is set.

Signed-off-by: Mark Wielaard <mjw@redhat.com>
---

diff --git a/tests/ChangeLog b/tests/ChangeLog
index 11974e15a..c549fa0bc 100644
--- a/tests/ChangeLog
+++ b/tests/ChangeLog
@@ -1,3 +1,7 @@
+2013-11-05  Mark Wielaard  <mjw@redhat.com>
+
+	* allfcts.c (main): Correct dwarf_getfuncs return value check.
+
 2013-10-10  Mark Wielaard  <mjw@redhat.com>
 	    Josh Stone  <jistone@redhat.com>
 
diff --git a/tests/allfcts.c b/tests/allfcts.c
index 7803722f5..10e0f07b8 100644
--- a/tests/allfcts.c
+++ b/tests/allfcts.c
@@ -63,7 +63,7 @@ main (int argc, char *argv[])
 		{
 		  doff = dwarf_getfuncs (die, cb, NULL, doff);
 		}
-	      while (doff > 0);
+	      while (doff != 0 && dwarf_errno () == 0);
 
 	      off = noff;
 	    }