public inbox for cygwin-cvs@sourceware.org
help / color / mirror / Atom feed
* [newlib-cygwin] Cygwin: Add addresses as module offsets in .stackdump file
@ 2022-10-29 17:34 Jon TURNEY
0 siblings, 0 replies; only message in thread
From: Jon TURNEY @ 2022-10-29 17:34 UTC (permalink / raw)
To: cygwin-cvs
https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=d59651d4daff0d31837ea852cd01404f8e10dca4
commit d59651d4daff0d31837ea852cd01404f8e10dca4
Author: Jon Turney <jon.turney@dronecode.org.uk>
Date: Fri Oct 28 12:26:56 2022 +0100
Cygwin: Add addresses as module offsets in .stackdump file
This adds an additional column to the stack trace in a .stackdump file,
which gives the stack frame return address as a module name+offset. This
makes it a possible to convert the address to a function name without
having to guess what module the address belongs to.
> Stack trace:
> Frame Function Args
> 0007FFFFCC30 0001004010E9 (000180048055, 000180046FA0, 000000000002, 00018031E160) segv-test.exe+0x10E9
> 0007FFFFCD30 0001800480C1 (000000000000, 000000000000, 000000000000, 000000000000) cygwin1.dll+0x80C1
> 0007FFFFFFF0 000180045C86 (000000000000, 000000000000, 000000000000, 000000000000) cygwin1.dll+0x5C86
> 0007FFFFFFF0 000180045D34 (000000000000, 000000000000, 000000000000, 000000000000) cygwin1.dll+0x5D34
> End of stack trace
Loosely based on this patch [1] by Brian Dessent.
[1] https://cygwin.com/pipermail/cygwin-patches/2008q1/006306.html
Diff:
---
winsup/cygwin/exceptions.cc | 30 +++++++++++++++++++++++++++++-
1 file changed, 29 insertions(+), 1 deletion(-)
diff --git a/winsup/cygwin/exceptions.cc b/winsup/cygwin/exceptions.cc
index a15bc16c5..1e9ea26bf 100644
--- a/winsup/cygwin/exceptions.cc
+++ b/winsup/cygwin/exceptions.cc
@@ -324,6 +324,34 @@ stack_info::walk ()
return 1;
}
+/*
+ Walk the list of modules in the current process to find the one containing
+ 'func_va'.
+
+ This implementation requires no allocation of memory and minimal system
+ calls, so it should be safe in the context of an exception handler.
+*/
+static char *
+prettyprint_va (PVOID func_va)
+{
+ static char buf[256];
+ buf[0] = '\0';
+
+ PLIST_ENTRY head = &NtCurrentTeb()->Peb->Ldr->InMemoryOrderModuleList;
+ for (PLIST_ENTRY x = head->Flink; x != head; x = x->Flink)
+ {
+ PLDR_DATA_TABLE_ENTRY mod = CONTAINING_RECORD (x, LDR_DATA_TABLE_ENTRY,
+ InMemoryOrderLinks);
+ if (mod->DllBase > func_va)
+ continue;
+
+ __small_sprintf (buf, "%S+0x%x", &mod->BaseDllName,
+ (DWORD_PTR)func_va - (DWORD_PTR)mod->DllBase);
+ }
+
+ return buf;
+}
+
void
cygwin_exception::dumpstack ()
{
@@ -350,7 +378,7 @@ cygwin_exception::dumpstack ()
for (unsigned j = 0; j < NPARAMS; j++)
small_printf ("%s%012X", j == 0 ? " (" : ", ",
thestack.sf.Params[j]);
- small_printf (")\r\n");
+ small_printf (") %s\r\n", prettyprint_va((PVOID)thestack.sf.AddrPC.Offset));
}
small_printf ("End of stack trace%s\r\n",
i == DUMPSTACK_FRAME_LIMIT ?
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2022-10-29 17:34 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-29 17:34 [newlib-cygwin] Cygwin: Add addresses as module offsets in .stackdump file Jon TURNEY
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).