This is the mail archive of the
cygwin-patches
mailing list for the Cygwin project.
[PATCH] Make ldd stop after any non-continuable exception
- From: Jon Turney <jon dot turney at dronecode dot org dot uk>
- To: cygwin-patches at cygwin dot com
- Cc: Jon Turney <jon dot turney at dronecode dot org dot uk>
- Date: Tue, 4 Apr 2017 18:51:10 +0100
- Subject: [PATCH] Make ldd stop after any non-continuable exception
- Authentication-results: sourceware.org; auth=none
Ensure that ldd always stops when the exception is flagged as
non-continuable.
Also arrange for ldd to exit with a non-zero exit code if something went
wrong which prevented us from listing all dynamic dependencies.
Signed-off-by: Jon Turney <jon.turney@dronecode.org.uk>
---
Notes:
I saw an instance of ldd which was continuously looping with the same
STATUS_DLL_NOT_FOUND exception being reported.
winsup/utils/ldd.cc | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/winsup/utils/ldd.cc b/winsup/utils/ldd.cc
index 8e891d8..bbc62f1 100644
--- a/winsup/utils/ldd.cc
+++ b/winsup/utils/ldd.cc
@@ -302,6 +302,9 @@ report (const char *in_fn, bool multiple)
dlls dll_list = {};
dlls *dll_last = &dll_list;
const wchar_t *process_fn = NULL;
+
+ int res = 0;
+
while (1)
{
bool exitnow = false;
@@ -356,6 +359,11 @@ report (const char *in_fn, bool multiple)
TerminateProcess (hProcess, 0);
break;
}
+ if (ev.u.Exception.ExceptionRecord.ExceptionFlags &
+ EXCEPTION_NONCONTINUABLE) {
+ res = 1;
+ goto print_and_exit;
+ }
break;
case EXIT_PROCESS_DEBUG_EVENT:
print_and_exit:
@@ -374,7 +382,7 @@ print_and_exit:
break;
}
- return 0;
+ return res;
}
int
--
2.8.3