I don't know how it is possible, but an #include directive was missing in
the patch I submitted. The attached patch fixes this. Can this patch please
be applied ?

Thanks,

Bart.

---------- Forwarded message ----------
From: <[EMAIL PROTECTED]>
Date: Jan 2, 2008 11:07 AM
Subject: [Valgrind-developers] valgrind: r7309 - trunk/exp-drd
To: [email protected]


Author: tom
Date: 2008-01-02 10:07:44 +0000 (Wed, 02 Jan 2008)
New Revision: 7309

Log:
Stop drd trying to get a backtrace when there is no stack pointer.

Patch from Bart Van Assche <[EMAIL PROTECTED]>.

Modified:
  trunk/exp-drd/drd_segment.c


Modified: trunk/exp-drd/drd_segment.c
===================================================================
--- trunk/exp-drd/drd_segment.c 2007-12-30 12:28:26 UTC (rev 7308)
+++ trunk/exp-drd/drd_segment.c 2008-01-02 10:07:44 UTC (rev 7309)
@@ -64,7 +64,7 @@
  sg->next = 0;
  sg->prev = 0;

-  if (vg_created != VG_INVALID_THREADID)
+  if (vg_created != VG_INVALID_THREADID && VG_(get_SP)(vg_created) != 0)
    sg->stacktrace = VG_(record_ExeContext)(vg_created, 0);
  else
    sg->stacktrace = 0;
Index: exp-drd/drd_segment.c
===================================================================
--- exp-drd/drd_segment.c	(revision 7313)
+++ exp-drd/drd_segment.c	(working copy)
@@ -31,6 +31,7 @@
 #include "pub_tool_libcassert.h"  // tl_assert()
 #include "pub_tool_libcbase.h"    // VG_(strlen)()
 #include "pub_tool_libcprint.h"   // VG_(printf)()
+#include "pub_tool_machine.h"     // VG_(get_SP)()
 #include "pub_tool_mallocfree.h"  // VG_(malloc)(), VG_(free)()
 #include "pub_tool_threadstate.h" // VG_INVALID_THREADID
 
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Valgrind-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/valgrind-developers

Reply via email to