Re: systemtap release 0.7 announcement

[Srinivasa DS <srinivasa at in dot ibm dot com>]

Frank Ch. Eigler wrote:
> Hi -
>
>
> The systemtap team is proud to announce release 0.7.  The previous
> release was version 0.6.2 from March.  There are several performance
> and capability improvements summarized below.  Users are encouraged to
> upgrade.

Hi Frank
 I hit segmentation fault while executing systemtap tests on x86_64. 
section name(secname) was not verified for the NULL before dereferencing 
it in dump_unwindsyms() and hence the problem.  Iam attaching the patch 
to avoid this problem.


Signed-off-by: Srinivasa DS <srinivasa@in.ibm.com>

---
 translate.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: systemtap-0.7/translate.cxx
===================================================================
--- systemtap-0.7.orig/translate.cxx
+++ systemtap-0.7/translate.cxx
@@ -4426,7 +4426,7 @@ dump_unwindsyms (Dwfl_Module *m,
                   secname = "_stext"; // not actually used
                   // NB: don't subtract session.sym_stext, which could 
be inconveniently NULL.
                 }
-              else if (n > 0 && strcmp (secname, ".text")) /* XXX: only 
care about .text-related relocations for now. */
+              else if (n > 0 && secname && strcmp (secname, ".text")) 
/* XXX: only care about .text-related relocations for now. */
                 {
                   if (c->session.verbose > 2)
                     clog << "Skipped symbol " << name << ", due to 
non-.text relocation section " << secname << endl;