Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Ulrich Weigand <weigand@i1.informatik.uni-erlangen.de>
To: gdb-patches@sources.redhat.com
Subject: [PATCH] Fix signull test case
Date: Thu, 20 May 2004 13:30:00 -0000	[thread overview]
Message-ID: <200405201330.PAA24916@faui1d.informatik.uni-erlangen.de> (raw)

Hello,

the signull.exp test case fails when running under a 2.6 kernel,
because it uses longjmp to leave a SIGSEGV handler.  This results
in the signal remaining blocked for the rest of program execution,
and the second segmentation violation triggered by the test case
simply terminates the program.  The fix is to use siglongjmp.

Also, the test case uses 'signal' to install the signal handler;
according to POSIX it is undefined whether this routine installs
the handler with the 'one-shot' property or not.  (Apparently,
on BSD-like systems 'one-shot' is the default.)

Thus, I've also changed the test case to use sigaction instead,
to make sure the handler remains active across multiple invocations.
(This isn't currently an actual problem on Linux systems -- if you
prefer I can remove this part.)

Tested on s390-ibm-linux and s390x-ibm-linux, fixes one test suite
failure.

OK?


ChangeLog:

	* gdb.bas/signull.c: Use sigsetjmp/siglongjmp instead of 
	setjmp/longjmp.  Use sigaction instead of signal.

Index: gdb/testsuite/gdb.base/signull.c
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.base/signull.c,v
retrieving revision 1.1
diff -c -p -r1.1 signull.c
*** gdb/testsuite/gdb.base/signull.c	13 May 2004 18:12:07 -0000	1.1
--- gdb/testsuite/gdb.base/signull.c	20 May 2004 13:10:52 -0000
***************
*** 19,24 ****
--- 19,25 ----
  
  #include <signal.h>
  #include <setjmp.h>
+ #include <stdlib.h>
  
  enum tests {
    code_entry_point, code_descriptor, data_pointer
*************** jmp_buf env;
*** 38,44 ****
  extern void
  keeper (int sig)
  {
!   longjmp (env, 0);
  }
  
  extern long
--- 39,45 ----
  extern void
  keeper (int sig)
  {
!   siglongjmp (env, 0);
  }
  
  extern long
*************** int
*** 59,68 ****
  main ()
  {
    static volatile int i;
!   signal (SIGSEGV, keeper);
    for (i = 0; i < 10; i++)
      {
!       setjmp (env);
        bowler ();
      }
  }
--- 60,74 ----
  main ()
  {
    static volatile int i;
! 
!   struct sigaction act;
!   memset (&act, 0, sizeof act);
!   act.sa_handler = keeper;
!   sigaction (SIGSEGV, &act, NULL);
! 
    for (i = 0; i < 10; i++)
      {
!       sigsetjmp (env, 1);
        bowler ();
      }
  }
-- 
  Dr. Ulrich Weigand
  weigand@informatik.uni-erlangen.de


             reply	other threads:[~2004-05-20 13:30 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-05-20 13:30 Ulrich Weigand [this message]
2004-05-20 14:05 ` Daniel Jacobowitz
2004-05-21 17:10 ` Andrew Cagney

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200405201330.PAA24916@faui1d.informatik.uni-erlangen.de \
    --to=weigand@i1.informatik.uni-erlangen.de \
    --cc=gdb-patches@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox