]> granicus.if.org Git - strace/blobdiff - signal.c
Fix "format not a string literal" warning caused by tprintf(str)
[strace] / signal.c
index 874e3d52c79cbe18954d6da56d5879ba37549d95..84af14f4cf98028056be0f6d3bbcfec72c627f30 100644 (file)
--- a/signal.c
+++ b/signal.c
@@ -35,6 +35,7 @@
 
 #include "defs.h"
 
+#include <stdint.h>
 #include <signal.h>
 #include <sys/user.h>
 #include <fcntl.h>
 
 #ifdef IA64
 # include <asm/ptrace_offsets.h>
-#endif /* !IA64 */
+#endif
 
-#if HAVE_ASM_REG_H
-# if defined (SPARC) || defined (SPARC64)
-#  define fpq kernel_fpq
-#  define fq kernel_fq
-#  define fpu kernel_fpu
-# endif
-# include <asm/reg.h>
-# if defined (SPARC) || defined (SPARC64)
-#  undef fpq
-#  undef fq
-#  undef fpu
-# endif
 #if defined (LINUX) && defined (SPARC64)
-# define r_pc r_tpc
 # undef PTRACE_GETREGS
 # define PTRACE_GETREGS PTRACE_GETREGS64
 # undef PTRACE_SETREGS
 # define PTRACE_SETREGS PTRACE_SETREGS64
 #endif /* LINUX && SPARC64 */
-#endif /* HAVE_ASM_REG_H */
 
-#if defined (SPARC) || defined (SPARC64)
-typedef struct {
-       struct regs             si_regs;
-       int                     si_mask;
-} m_siginfo_t;
-#elif defined (MIPS)
+#if defined (SPARC) || defined (SPARC64) || defined (MIPS)
 typedef struct {
        struct pt_regs          si_regs;
        int                     si_mask;
@@ -133,7 +115,7 @@ struct sigcontext_struct {
        unsigned long cr2;
 };
 #else /* !I386 */
-#ifdef M68K
+#if defined M68K && !defined HAVE_STRUCT_SIGCONTEXT
 struct sigcontext
 {
        unsigned long sc_mask;
@@ -149,36 +131,18 @@ struct sigcontext
 #endif /* M68K */
 #endif /* !I386 */
 #endif /* !HAVE_ASM_SIGCONTEXT_H */
+
 #ifndef NSIG
+#warning: NSIG is not defined, using 32
 #define NSIG 32
 #endif
 #ifdef ARM
+/* Ugh. Is this really correct? ARM has no RT signals?! */
 #undef NSIG
 #define NSIG 32
 #endif
-#endif /* LINUX */
-
-const char *const signalent0[] = {
-#include "signalent.h"
-};
-const int nsignals0 = sizeof signalent0 / sizeof signalent0[0];
 
-#if SUPPORTED_PERSONALITIES >= 2
-const char *const signalent1[] = {
-#include "signalent1.h"
-};
-const int nsignals1 = sizeof signalent1 / sizeof signalent1[0];
-#endif /* SUPPORTED_PERSONALITIES >= 2 */
-
-#if SUPPORTED_PERSONALITIES >= 3
-const char *const signalent2[] = {
-#include "signalent2.h"
-};
-const int nsignals2 = sizeof signalent2 / sizeof signalent2[0];
-#endif /* SUPPORTED_PERSONALITIES >= 3 */
-
-const char *const *signalent;
-int nsignals;
+#endif /* LINUX */
 
 #if defined(SUNOS4) || defined(FREEBSD)
 
@@ -196,7 +160,7 @@ static const struct xlat sigvec_flags[] = {
 
 #if defined LINUX && (defined I386 || defined X86_64)
 /* The libc headers do not define this constant since it should only be
-   used by the implementation.  So wwe define it here.  */
+   used by the implementation.  So we define it here.  */
 # ifndef SA_RESTORER
 #  define SA_RESTORER 0x04000000
 # endif
@@ -215,10 +179,16 @@ static const struct xlat sigact_flags[] = {
 #ifdef SA_INTERRUPT
        { SA_INTERRUPT, "SA_INTERRUPT"  },
 #endif
-#ifdef SA_NOMASK
+#ifdef SA_NODEFER
+       { SA_NODEFER,   "SA_NODEFER"    },
+#endif
+#if defined SA_NOMASK && SA_NODEFER != SA_NOMASK
        { SA_NOMASK,    "SA_NOMASK"     },
 #endif
-#ifdef SA_ONESHOT
+#ifdef SA_RESETHAND
+       { SA_RESETHAND, "SA_RESETHAND"  },
+#endif
+#if defined SA_ONESHOT && SA_ONESHOT != SA_RESETHAND
        { SA_ONESHOT,   "SA_ONESHOT"    },
 #endif
 #ifdef SA_SIGINFO
@@ -241,6 +211,9 @@ static const struct xlat sigact_flags[] = {
 #endif
 #ifdef _SA_BSDCALL
        { _SA_BSDCALL,  "_SA_BSDCALL"   },
+#endif
+#ifdef SA_NOPTRACE
+       { SA_NOPTRACE,  "SA_NOPTRACE"   },
 #endif
        { 0,            NULL            },
 };
@@ -267,29 +240,48 @@ static const struct xlat sigprocmaskcmds[] = {
 #endif
 #endif
 
+/* Note on the size of sigset_t:
+ *
+ * In glibc, sigset_t is an array with space for 1024 bits (!),
+ * even though all arches supported by Linux have only 64 signals
+ * except MIPS, which has 128. IOW, it is 128 bytes long.
+ *
+ * In-kernel sigset_t is sized correctly (it is either 64 or 128 bit long).
+ * However, some old syscall return only 32 lower bits (one word).
+ * Example: sys_sigpending vs sys_rt_sigpending.
+ *
+ * Be aware of this fact when you try to
+ *     memcpy(&tcp->u_arg[1], &something, sizeof(sigset_t))
+ * - sizeof(sigset_t) is much bigger than you think,
+ * it may overflow tcp->u_arg[] array, and it may try to copy more data
+ * than is really available in <something>.
+ * Similarly,
+ *     umoven(tcp, addr, sizeof(sigset_t), &sigset)
+ * may be a bad idea: it'll try to read much more data than needed
+ * to fetch a sigset_t.
+ * Use (NSIG / 8) as a size instead.
+ */
+
 const char *
-signame(sig)
-int sig;
+signame(int sig)
 {
-       static char buf[30];
-       if (sig >= 0 && sig < nsignals) {
+       static char buf[sizeof("SIGRT_%d") + sizeof(int)*3];
+
+       if (sig >= 0 && sig < nsignals)
                return signalent[sig];
 #ifdef SIGRTMIN
-       } else if (sig >= __SIGRTMIN && sig <= __SIGRTMAX) {
-               sprintf(buf, "SIGRT_%ld", (long)(sig - __SIGRTMIN));
-               return buf;
-#endif /* SIGRTMIN */
-       } else {
-               sprintf(buf, "%d", sig);
+       if (sig >= __SIGRTMIN && sig <= __SIGRTMAX) {
+               sprintf(buf, "SIGRT_%d", (int)(sig - __SIGRTMIN));
                return buf;
        }
+#endif
+       sprintf(buf, "%d", sig);
+       return buf;
 }
 
 #ifndef UNIXWARE
 static void
-long_to_sigset(l, s)
-long l;
-sigset_t *s;
+long_to_sigset(long l, sigset_t *s)
 {
        sigemptyset(s);
        *(long *)s = l;
@@ -297,11 +289,7 @@ sigset_t *s;
 #endif
 
 static int
-copy_sigset_len(tcp, addr, s, len)
-struct tcb *tcp;
-long addr;
-sigset_t *s;
-int len;
+copy_sigset_len(struct tcb *tcp, long addr, sigset_t *s, int len)
 {
        if (len > sizeof(*s))
                len = sizeof(*s);
@@ -322,10 +310,21 @@ static const char *
 sprintsigmask(const char *str, sigset_t *mask, int rt)
 /* set might include realtime sigs */
 {
+       /* Was [8 * sizeof(sigset_t) * 8], but
+        * glibc sigset_t is huge (1024 bits = 128 *bytes*),
+        * and we were ending up with 8k (!) buffer here.
+        *
+        * No Unix system can have sig > 255
+        * (waitpid API won't be able to indicate death from one)
+        * and sig 0 doesn't exist either.
+        * Therefore max possible no of sigs is 255: 1..255
+        */
+       static char outstr[8 * 255];
+
        int i, nsigs;
        int maxsigs;
-       char *format, *s;
-       static char outstr[8 * sizeof(sigset_t) * 8];
+       const char *format;
+       char *s;
 
        strcpy(outstr, str);
        s = outstr + strlen(outstr);
@@ -384,18 +383,15 @@ sprintsigmask(const char *str, sigset_t *mask, int rt)
 }
 
 static void
-printsigmask(mask, rt)
-sigset_t *mask;
-int rt;
+printsigmask(sigset_t *mask, int rt)
 {
-       tprintf("%s", sprintsigmask("", mask, rt));
+       tprints(sprintsigmask("", mask, rt));
 }
 
 void
-printsignal(nr)
-int nr;
+printsignal(int nr)
 {
-       tprintf(signame(nr));
+       tprints(signame(nr));
 }
 
 void
@@ -449,14 +445,19 @@ print_sigset(struct tcb *tcp, long addr, int rt)
 #define POLL_ERR        4       /* i/o error */
 #define POLL_PRI        5       /* high priority input available */
 #define POLL_HUP        6       /* device disconnected */
+#define SI_KERNEL      0x80    /* sent by kernel */
 #define SI_USER         0       /* sent by kill, sigsend, raise */
 #define SI_QUEUE        -1      /* sent by sigqueue */
 #define SI_TIMER        -2      /* sent by timer expiration */
 #define SI_MESGQ        -3      /* sent by real time mesq state change */
 #define SI_ASYNCIO      -4      /* sent by AIO completion */
-#define SI_SIGIO       -5      /* Sent by SIGIO */
-#define SI_TKILL       -6      /* Sent by tkill */
-#endif
+#define SI_SIGIO       -5      /* sent by SIGIO */
+#define SI_TKILL       -6      /* sent by tkill */
+#define SI_ASYNCNL     -60     /* sent by asynch name lookup completion */
+
+#define SI_FROMUSER(sip)       ((sip)->si_code <= 0)
+
+#endif /* LINUX */
 
 #if __GLIBC_MINOR__ < 1
 /* Type for data associated with a signal.  */
@@ -467,7 +468,7 @@ typedef union sigval
 } sigval_t;
 
 # define __SI_MAX_SIZE     128
-# define __SI_PAD_SIZE     ((__SI_MAX_SIZE / sizeof (int)) - 3)
+# define __SI_PAD_SIZE     ((__SI_MAX_SIZE / sizeof(int)) - 3)
 
 typedef struct siginfo
 {
@@ -545,32 +546,38 @@ typedef struct siginfo
 #if defined (SVR4) || defined (LINUX)
 
 static const struct xlat siginfo_codes[] = {
-#ifdef SI_NOINFO
-       { SI_NOINFO,    "SI_NOINFO"     },
+#ifdef SI_KERNEL
+       { SI_KERNEL,    "SI_KERNEL"     },
 #endif
 #ifdef SI_USER
        { SI_USER,      "SI_USER"       },
 #endif
-#ifdef SI_LWP
-       { SI_LWP,       "SI_LWP"        },
-#endif
 #ifdef SI_QUEUE
        { SI_QUEUE,     "SI_QUEUE"      },
 #endif
 #ifdef SI_TIMER
        { SI_TIMER,     "SI_TIMER"      },
 #endif
-#ifdef SI_ASYNCIO
-       { SI_ASYNCIO,   "SI_ASYNCIO"    },
-#endif
 #ifdef SI_MESGQ
        { SI_MESGQ,     "SI_MESGQ"      },
 #endif
+#ifdef SI_ASYNCIO
+       { SI_ASYNCIO,   "SI_ASYNCIO"    },
+#endif
 #ifdef SI_SIGIO
        { SI_SIGIO,     "SI_SIGIO"      },
 #endif
 #ifdef SI_TKILL
        { SI_TKILL,     "SI_TKILL"      },
+#endif
+#ifdef SI_ASYNCNL
+       { SI_ASYNCNL,   "SI_ASYNCNL"    },
+#endif
+#ifdef SI_NOINFO
+       { SI_NOINFO,    "SI_NOINFO"     },
+#endif
+#ifdef SI_LWP
+       { SI_LWP,       "SI_LWP"        },
 #endif
        { 0,            NULL            },
 };
@@ -655,14 +662,12 @@ static const struct xlat sigbus_codes[] = {
 };
 
 void
-printsiginfo(sip, verbose)
-siginfo_t *sip;
-int verbose;
+printsiginfo(siginfo_t *sip, int verbose)
 {
        const char *code;
 
        if (sip->si_signo == 0) {
-               tprintf ("{}");
+               tprintf("{}");
                return;
        }
        tprintf("{si_signo=");
@@ -718,23 +723,36 @@ int verbose;
                }
 #ifdef SI_FROMUSER
                if (SI_FROMUSER(sip)) {
-                       tprintf(", si_pid=%ld, si_uid=%ld",
-                               sip->si_pid, sip->si_uid);
-#ifdef SI_QUEUE
+                       tprintf(", si_pid=%lu, si_uid=%lu",
+                               (unsigned long) sip->si_pid,
+                               (unsigned long) sip->si_uid);
                        switch (sip->si_code) {
-                       case SI_QUEUE:
+#ifdef SI_USER
+                       case SI_USER:
+                               break;
+#endif
+#ifdef SI_TKILL
+                       case SI_TKILL:
+                               break;
+#endif
 #ifdef SI_TIMER
                        case SI_TIMER:
-#endif /* SI_QUEUE */
-                       case SI_ASYNCIO:
-#ifdef SI_MESGQ
-                       case SI_MESGQ:
-#endif /* SI_MESGQ */
-                               tprintf(", si_value=%d",
-                                       sip->si_value.sival_int);
+                               tprintf(", si_value=%d", sip->si_int);
                                break;
+#endif
+#ifdef LINUX
+                       default:
+                               if (!sip->si_ptr)
+                                       break;
+                               if (!verbose)
+                                       tprintf(", ...");
+                               else
+                                       tprintf(", si_value={int=%u, ptr=%#lx}",
+                                               sip->si_int,
+                                               (unsigned long) sip->si_ptr);
+                               break;
+#endif
                        }
-#endif /* SI_QUEUE */
                }
                else
 #endif /* SI_FROMUSER */
@@ -771,13 +789,16 @@ int verbose;
                                break;
 #ifdef LINUX
                        default:
-                               tprintf(", si_pid=%lu, si_uid=%lu, ",
-                                       (unsigned long) sip->si_pid,
-                                       (unsigned long) sip->si_uid);
+                               if (sip->si_pid || sip->si_uid)
+                                       tprintf(", si_pid=%lu, si_uid=%lu",
+                                               (unsigned long) sip->si_pid,
+                                               (unsigned long) sip->si_uid);
+                               if (!sip->si_ptr)
+                                       break;
                                if (!verbose)
-                                       tprintf("...");
+                                       tprintf("...");
                                else {
-                                       tprintf("si_value={int=%u, ptr=%#lx}",
+                                       tprintf("si_value={int=%u, ptr=%#lx}",
                                                sip->si_int,
                                                (unsigned long) sip->si_ptr);
                                }
@@ -791,146 +812,10 @@ int verbose;
 
 #endif /* SVR4 || LINUX */
 
-#ifdef LINUX
-
-static void
-parse_sigset_t (const char *str, sigset_t *set)
-{
-       const char *p;
-       unsigned int digit;
-       int i;
-
-       sigemptyset(set);
-
-       p = strchr(str, '\n');
-       if (p == NULL)
-               p = strchr(str, '\0');
-       for (i = 0; p-- > str; i += 4) {
-               if (*p >= '0' && *p <= '9')
-                       digit = *p - '0';
-               else if (*p >= 'a' && *p <= 'f')
-                       digit = *p - 'a' + 10;
-               else if (*p >= 'A' && *p <= 'F')
-                       digit = *p - 'A' + 10;
-               else
-                       break;
-               if (digit & 1)
-                       sigaddset(set, i + 1);
-               if (digit & 2)
-                       sigaddset(set, i + 2);
-               if (digit & 4)
-                       sigaddset(set, i + 3);
-               if (digit & 8)
-                       sigaddset(set, i + 4);
-       }
-}
-
-#endif
-
-/*
- * Check process TCP for the disposition of signal SIG.
- * Return 1 if the process would somehow manage to  survive signal SIG,
- * else return 0.  This routine will never be called with SIGKILL.
- */
-int
-sigishandled(tcp, sig)
-struct tcb *tcp;
-int sig;
-{
-#ifdef LINUX
-       int sfd;
-       char sname[32];
-       char buf[2048];
-       char *s;
-       int i;
-       sigset_t ignored, caught;
-#endif
-#ifdef SVR4
-       /*
-        * Since procfs doesn't interfere with wait I think it is safe
-        * to punt on this question.  If not, the information is there.
-        */
-       return 1;
-#else /* !SVR4 */
-       switch (sig) {
-       case SIGCONT:
-       case SIGSTOP:
-       case SIGTSTP:
-       case SIGTTIN:
-       case SIGTTOU:
-       case SIGCHLD:
-       case SIGIO:
-#if defined(SIGURG) && SIGURG != SIGIO
-       case SIGURG:
-#endif
-       case SIGWINCH:
-               /* Gloria Gaynor says ... */
-               return 1;
-       default:
-               break;
-       }
-#endif /* !SVR4 */
-#ifdef LINUX
-
-       /* This is incredibly costly but it's worth it. */
-       /* NOTE: LinuxThreads internally uses SIGRTMIN, SIGRTMIN + 1 and
-          SIGRTMIN + 2, so we can't use the obsolete /proc/%d/stat which
-          doesn't handle real-time signals). */
-       sprintf(sname, "/proc/%d/status", tcp->pid);
-       if ((sfd = open(sname, O_RDONLY)) == -1) {
-               perror(sname);
-               return 1;
-       }
-       i = read(sfd, buf, sizeof(buf));
-       buf[i] = '\0';
-       close(sfd);
-       /*
-        * Skip the extraneous fields. We need to skip
-        * command name has any spaces in it.  So be it.
-        */
-       s = strstr(buf, "SigIgn:\t");
-       if (!s)
-       {
-               fprintf(stderr, "/proc/pid/status format error\n");
-               return 1;
-       }
-       parse_sigset_t(s + 8, &ignored);
-
-       s = strstr(buf, "SigCgt:\t");
-       if (!s)
-       {
-               fprintf(stderr, "/proc/pid/status format error\n");
-               return 1;
-       }
-       parse_sigset_t(s + 8, &caught);
-
-#ifdef DEBUG
-       fprintf(stderr, "sigs: %016qx %016qx (sig=%d)\n",
-               *(long long *) &ignored, *(long long *) &caught, sig);
-#endif
-       if (sigismember(&ignored, sig) || sigismember(&caught, sig))
-               return 1;
-#endif /* LINUX */
-
-#ifdef SUNOS4
-       void (*u_signal)();
-
-       if (upeek(tcp->pid, uoff(u_signal[0]) + sig*sizeof(u_signal),
-           (long *) &u_signal) < 0) {
-               return 0;
-       }
-       if (u_signal != SIG_DFL)
-               return 1;
-#endif /* SUNOS4 */
-
-       return 0;
-}
-
 #if defined(SUNOS4) || defined(FREEBSD)
 
 int
-sys_sigvec(tcp)
-struct tcb *tcp;
+sys_sigvec(struct tcb *tcp)
 {
        struct sigvec sv;
        long addr;
@@ -988,8 +873,7 @@ struct tcb *tcp;
 }
 
 int
-sys_sigpause(tcp)
-struct tcb *tcp;
+sys_sigpause(struct tcb *tcp)
 {
        if (entering(tcp)) {    /* WTA: UD had a bug here: he forgot the braces */
                sigset_t sigm;
@@ -1000,8 +884,7 @@ struct tcb *tcp;
 }
 
 int
-sys_sigstack(tcp)
-struct tcb *tcp;
+sys_sigstack(struct tcb *tcp)
 {
        struct sigstack ss;
        long addr;
@@ -1024,8 +907,7 @@ struct tcb *tcp;
 }
 
 int
-sys_sigcleanup(tcp)
-struct tcb *tcp;
+sys_sigcleanup(struct tcb *tcp)
 {
        return 0;
 }
@@ -1035,8 +917,7 @@ struct tcb *tcp;
 #ifndef SVR4
 
 int
-sys_sigsetmask(tcp)
-struct tcb *tcp;
+sys_sigsetmask(struct tcb *tcp)
 {
        if (entering(tcp)) {
                sigset_t sigm;
@@ -1063,8 +944,7 @@ struct tcb *tcp;
 
 #if defined(SUNOS4) || defined(FREEBSD)
 int
-sys_sigblock(tcp)
-struct tcb *tcp;
+sys_sigblock(struct tcb *tcp)
 {
        return sys_sigsetmask(tcp);
 }
@@ -1089,8 +969,7 @@ struct old_sigaction {
 #endif
 
 int
-sys_sigaction(tcp)
-struct tcb *tcp;
+sys_sigaction(struct tcb *tcp)
 {
        long addr;
 #ifdef LINUX
@@ -1114,23 +993,28 @@ struct tcb *tcp;
        else if (umove(tcp, addr, &sa) < 0)
                tprintf("{...}");
        else {
-               switch ((long) sa.SA_HANDLER) {
-               case (long) SIG_ERR:
-                       tprintf("{SIG_ERR}");
-                       break;
-               case (long) SIG_DFL:
-                       tprintf("{SIG_DFL}");
-                       break;
-               case (long) SIG_IGN:
+               /* Architectures using function pointers, like
+                * hppa, may need to manipulate the function pointer
+                * to compute the result of a comparison. However,
+                * the SA_HANDLER function pointer exists only in
+                * the address space of the traced process, and can't
+                * be manipulated by strace. In order to prevent the
+                * compiler from generating code to manipulate
+                * SA_HANDLER we cast the function pointers to long. */
+               if ((long)sa.SA_HANDLER == (long)SIG_ERR)
+                       tprintf("{SIG_ERR, ");
+               else if ((long)sa.SA_HANDLER == (long)SIG_DFL)
+                       tprintf("{SIG_DFL, ");
+               else if ((long)sa.SA_HANDLER == (long)SIG_IGN) {
 #ifndef USE_PROCFS
                        if (tcp->u_arg[0] == SIGTRAP) {
                                tcp->flags |= TCB_SIGTRAPPED;
                                kill(tcp->pid, SIGSTOP);
                        }
 #endif /* !USE_PROCFS */
-                       tprintf("{SIG_IGN}");
-                       break;
-               default:
+                       tprintf("{SIG_IGN");
+               }
+               else {
 #ifndef USE_PROCFS
                        if (tcp->u_arg[0] == SIGTRAP) {
                                tcp->flags |= TCB_SIGTRAPPED;
@@ -1139,7 +1023,7 @@ struct tcb *tcp;
 #endif /* !USE_PROCFS */
                        tprintf("{%#lx, ", (long) sa.SA_HANDLER);
 #ifndef LINUX
-                       printsigmask (&sa.sa_mask, 0);
+                       printsigmask(&sa.sa_mask, 0);
 #else
                        long_to_sigset(sa.sa_mask, &sigset);
                        printsigmask(&sigset, 0);
@@ -1163,20 +1047,19 @@ struct tcb *tcp;
 }
 
 int
-sys_signal(tcp)
-struct tcb *tcp;
+sys_signal(struct tcb *tcp)
 {
        if (entering(tcp)) {
                printsignal(tcp->u_arg[0]);
                tprintf(", ");
                switch (tcp->u_arg[1]) {
-               case (int) SIG_ERR:
+               case (long) SIG_ERR:
                        tprintf("SIG_ERR");
                        break;
-               case (int) SIG_DFL:
+               case (long) SIG_DFL:
                        tprintf("SIG_DFL");
                        break;
-               case (int) SIG_IGN:
+               case (long) SIG_IGN:
 #ifndef USE_PROCFS
                        if (tcp->u_arg[0] == SIGTRAP) {
                                tcp->flags |= TCB_SIGTRAPPED;
@@ -1196,25 +1079,25 @@ struct tcb *tcp;
                }
                return 0;
        }
-       else {
+       else if (!syserror(tcp)) {
                switch (tcp->u_rval) {
-                   case (int) SIG_ERR:
+                   case (long) SIG_ERR:
                        tcp->auxstr = "SIG_ERR"; break;
-                   case (int) SIG_DFL:
+                   case (long) SIG_DFL:
                        tcp->auxstr = "SIG_DFL"; break;
-                   case (int) SIG_IGN:
+                   case (long) SIG_IGN:
                        tcp->auxstr = "SIG_IGN"; break;
                    default:
                        tcp->auxstr = NULL;
                }
                return RVAL_HEX | RVAL_STR;
        }
+       return 0;
 }
 
 #ifdef SVR4
 int
-sys_sighold(tcp)
-struct tcb *tcp;
+sys_sighold(struct tcb *tcp)
 {
        if (entering(tcp)) {
                printsignal(tcp->u_arg[0]);
@@ -1228,62 +1111,54 @@ struct tcb *tcp;
 #ifdef LINUX
 
 int
-sys_sigreturn(tcp)
-struct tcb *tcp;
+sys_sigreturn(struct tcb *tcp)
 {
-#ifdef ARM
-       struct pt_regs regs;
-       struct sigcontext_struct sc;
-
+#if defined(ARM)
        if (entering(tcp)) {
+               struct pt_regs regs;
+               struct sigcontext_struct sc;
                tcp->u_arg[0] = 0;
-
                if (ptrace(PTRACE_GETREGS, tcp->pid, NULL, (void *)&regs) == -1)
                        return 0;
-
                if (umove(tcp, regs.ARM_sp, &sc) < 0)
                        return 0;
-
                tcp->u_arg[0] = 1;
                tcp->u_arg[1] = sc.oldmask;
        } else {
                sigset_t sigm;
-               long_to_sigset(tcp->u_arg[1], &sigm);
                tcp->u_rval = tcp->u_error = 0;
                if (tcp->u_arg[0] == 0)
                        return 0;
+               long_to_sigset(tcp->u_arg[1], &sigm);
                tcp->auxstr = sprintsigmask("mask now ", &sigm, 0);
                return RVAL_NONE | RVAL_STR;
        }
        return 0;
 #elif defined(S390) || defined(S390X)
-       long usp;
-       struct sigcontext_struct sc;
-
        if (entering(tcp)) {
+               long usp;
+               struct sigcontext_struct sc;
                tcp->u_arg[0] = 0;
-               if (upeek(tcp->pid,PT_GPR15,&usp)<0)
+               if (upeek(tcp, PT_GPR15, &usp) < 0)
                        return 0;
                if (umove(tcp, usp+__SIGNAL_FRAMESIZE, &sc) < 0)
                        return 0;
                tcp->u_arg[0] = 1;
-               memcpy(&tcp->u_arg[1],&sc.oldmask[0],sizeof(sigset_t));
+               memcpy(&tcp->u_arg[1], &sc.oldmask[0], NSIG / 8);
        } else {
                tcp->u_rval = tcp->u_error = 0;
                if (tcp->u_arg[0] == 0)
                        return 0;
-               tcp->auxstr = sprintsigmask("mask now ",(sigset_t *)&tcp->u_arg[1],0);
+               tcp->auxstr = sprintsigmask("mask now ", (sigset_t *)&tcp->u_arg[1], 0);
                return RVAL_NONE | RVAL_STR;
        }
        return 0;
-#else
-#ifdef I386
-       long esp;
-       struct sigcontext_struct sc;
-
+#elif defined(I386)
        if (entering(tcp)) {
+               long esp;
+               struct sigcontext_struct sc;
                tcp->u_arg[0] = 0;
-               if (upeek(tcp->pid, 4*UESP, &esp) < 0)
+               if (upeek(tcp, 4*UESP, &esp) < 0)
                        return 0;
                if (umove(tcp, esp, &sc) < 0)
                        return 0;
@@ -1292,50 +1167,55 @@ struct tcb *tcp;
        }
        else {
                sigset_t sigm;
-               long_to_sigset(tcp->u_arg[1], &sigm);
                tcp->u_rval = tcp->u_error = 0;
                if (tcp->u_arg[0] == 0)
                        return 0;
+               long_to_sigset(tcp->u_arg[1], &sigm);
                tcp->auxstr = sprintsigmask("mask now ", &sigm, 0);
                return RVAL_NONE | RVAL_STR;
        }
        return 0;
-#else /* !I386 */
-#ifdef IA64
-       struct sigcontext sc;
-       long sp;
-
+#elif defined(IA64)
        if (entering(tcp)) {
+               struct sigcontext sc;
+               long sp;
                /* offset of sigcontext in the kernel's sigframe structure: */
 #              define SIGFRAME_SC_OFFSET       0x90
                tcp->u_arg[0] = 0;
-               if (upeek(tcp->pid, PT_R12, &sp) < 0)
+               if (upeek(tcp, PT_R12, &sp) < 0)
                        return 0;
                if (umove(tcp, sp + 16 + SIGFRAME_SC_OFFSET, &sc) < 0)
                        return 0;
                tcp->u_arg[0] = 1;
-               memcpy(tcp->u_arg + 1, &sc.sc_mask, sizeof(sc.sc_mask));
+               memcpy(tcp->u_arg + 1, &sc.sc_mask, NSIG / 8);
        }
        else {
                sigset_t sigm;
-
-               memcpy(&sigm, tcp->u_arg + 1, sizeof (sigm));
                tcp->u_rval = tcp->u_error = 0;
                if (tcp->u_arg[0] == 0)
                        return 0;
+               sigemptyset(&sigm);
+               memcpy(&sigm, tcp->u_arg + 1, NSIG / 8);
                tcp->auxstr = sprintsigmask("mask now ", &sigm, 0);
                return RVAL_NONE | RVAL_STR;
        }
        return 0;
-#else /* !IA64 */
-#ifdef POWERPC
-       long esp;
-       struct sigcontext_struct sc;
-
+#elif defined(POWERPC)
        if (entering(tcp)) {
+               long esp;
+               struct sigcontext_struct sc;
                tcp->u_arg[0] = 0;
-               if (upeek(tcp->pid, sizeof(unsigned long)*PT_R1, &esp) < 0)
+               if (upeek(tcp, sizeof(unsigned long)*PT_R1, &esp) < 0)
                        return 0;
+               /* Skip dummy stack frame. */
+#ifdef POWERPC64
+               if (current_personality == 0)
+                       esp += 128;
+               else
+                       esp += 64;
+#else
+               esp += 64;
+#endif
                if (umove(tcp, esp, &sc) < 0)
                        return 0;
                tcp->u_arg[0] = 1;
@@ -1343,22 +1223,20 @@ struct tcb *tcp;
        }
        else {
                sigset_t sigm;
-               long_to_sigset(tcp->u_arg[1], &sigm);
                tcp->u_rval = tcp->u_error = 0;
                if (tcp->u_arg[0] == 0)
                        return 0;
+               long_to_sigset(tcp->u_arg[1], &sigm);
                tcp->auxstr = sprintsigmask("mask now ", &sigm, 0);
                return RVAL_NONE | RVAL_STR;
        }
        return 0;
-#else /* !POWERPC */
-#ifdef M68K
-       long usp;
-       struct sigcontext sc;
-
+#elif defined(M68K)
        if (entering(tcp)) {
+               long usp;
+               struct sigcontext sc;
                tcp->u_arg[0] = 0;
-               if (upeek(tcp->pid, 4*PT_USP, &usp) < 0)
+               if (upeek(tcp, 4*PT_USP, &usp) < 0)
                        return 0;
                if (umove(tcp, usp, &sc) < 0)
                        return 0;
@@ -1367,22 +1245,20 @@ struct tcb *tcp;
        }
        else {
                sigset_t sigm;
-               long_to_sigset(tcp->u_arg[1], &sigm);
                tcp->u_rval = tcp->u_error = 0;
                if (tcp->u_arg[0] == 0)
                        return 0;
+               long_to_sigset(tcp->u_arg[1], &sigm);
                tcp->auxstr = sprintsigmask("mask now ", &sigm, 0);
                return RVAL_NONE | RVAL_STR;
        }
        return 0;
-#else /* !M68K */
-#ifdef ALPHA
-       long fp;
-       struct sigcontext_struct sc;
-
+#elif defined(ALPHA)
        if (entering(tcp)) {
+               long fp;
+               struct sigcontext_struct sc;
                tcp->u_arg[0] = 0;
-               if (upeek(tcp->pid, REG_FP, &fp) < 0)
+               if (upeek(tcp, REG_FP, &fp) < 0)
                        return 0;
                if (umove(tcp, fp, &sc) < 0)
                        return 0;
@@ -1391,92 +1267,159 @@ struct tcb *tcp;
        }
        else {
                sigset_t sigm;
-               long_to_sigset(tcp->u_arg[1], &sigm);
                tcp->u_rval = tcp->u_error = 0;
                if (tcp->u_arg[0] == 0)
                        return 0;
+               long_to_sigset(tcp->u_arg[1], &sigm);
                tcp->auxstr = sprintsigmask("mask now ", &sigm, 0);
                return RVAL_NONE | RVAL_STR;
        }
        return 0;
-#else
-#if defined (SPARC) || defined (SPARC64)
-       long i1;
-       struct regs regs;
-       m_siginfo_t si;
-
-       if(ptrace(PTRACE_GETREGS, tcp->pid, (char *)&regs, 0) < 0) {
-               perror("sigreturn: PTRACE_GETREGS ");
-               return 0;
-       }
-       if(entering(tcp)) {
+#elif defined (SPARC) || defined (SPARC64)
+       if (entering(tcp)) {
+               long i1;
+               struct pt_regs regs;
+               m_siginfo_t si;
                tcp->u_arg[0] = 0;
-               i1 = regs.r_o1;
-               if(umove(tcp, i1, &si) < 0) {
-                       perror("sigreturn: umove ");
+               if (ptrace(PTRACE_GETREGS, tcp->pid, (char *)&regs, 0) < 0) {
+                       perror("sigreturn: PTRACE_GETREGS");
+                       return 0;
+               }
+               i1 = regs.u_regs[U_REG_O1];
+               if (umove(tcp, i1, &si) < 0) {
+                       perror("sigreturn: umove");
                        return 0;
                }
                tcp->u_arg[0] = 1;
                tcp->u_arg[1] = si.si_mask;
        } else {
                sigset_t sigm;
-               long_to_sigset(tcp->u_arg[1], &sigm);
                tcp->u_rval = tcp->u_error = 0;
-               if(tcp->u_arg[0] == 0)
+               if (tcp->u_arg[0] == 0)
                        return 0;
+               long_to_sigset(tcp->u_arg[1], &sigm);
                tcp->auxstr = sprintsigmask("mask now ", &sigm, 0);
                return RVAL_NONE | RVAL_STR;
        }
        return 0;
-#else
-#if defined (LINUX_MIPSN32) || defined (LINUX_MIPSN64)
+#elif defined (LINUX_MIPSN32) || defined (LINUX_MIPSN64)
        /* This decodes rt_sigreturn.  The 64-bit ABIs do not have
           sigreturn.  */
-       long sp;
-       struct ucontext uc;
-
-       if(entering(tcp)) {
-               tcp->u_arg[0] = 0;
-               if (upeek(tcp->pid, REG_SP, &sp) < 0)
-                       return 0;
+       if (entering(tcp)) {
+               long sp;
+               struct ucontext uc;
+               tcp->u_arg[0] = 0;
+               if (upeek(tcp, REG_SP, &sp) < 0)
+                       return 0;
                /* There are six words followed by a 128-byte siginfo.  */
                sp = sp + 6 * 4 + 128;
                if (umove(tcp, sp, &uc) < 0)
-                       return 0;
+                       return 0;
                tcp->u_arg[0] = 1;
                tcp->u_arg[1] = *(long *) &uc.uc_sigmask;
        } else {
                sigset_t sigm;
+               tcp->u_rval = tcp->u_error = 0;
+               if (tcp->u_arg[0] == 0)
+                       return 0;
                long_to_sigset(tcp->u_arg[1], &sigm);
-               tcp->u_rval = tcp->u_error = 0;
-               if(tcp->u_arg[0] == 0)
-                       return 0;
                tcp->auxstr = sprintsigmask("mask now ", &sigm, 0);
                return RVAL_NONE | RVAL_STR;
        }
        return 0;
-#else
-#ifdef MIPS
-       long sp;
-       struct pt_regs regs;
-       m_siginfo_t si;
-
-       if(ptrace(PTRACE_GETREGS, tcp->pid, (char *)&regs, 0) < 0) {
-               perror("sigreturn: PTRACE_GETREGS ");
-               return 0;
-       }
-       if(entering(tcp)) {
-               tcp->u_arg[0] = 0;
+#elif defined(MIPS)
+       if (entering(tcp)) {
+               long sp;
+               struct pt_regs regs;
+               m_siginfo_t si;
+               tcp->u_arg[0] = 0;
+               if (ptrace(PTRACE_GETREGS, tcp->pid, (char *)&regs, 0) < 0) {
+                       perror("sigreturn: PTRACE_GETREGS");
+                       return 0;
+               }
                sp = regs.regs[29];
                if (umove(tcp, sp, &si) < 0)
+                       return 0;
                tcp->u_arg[0] = 1;
                tcp->u_arg[1] = si.si_mask;
        } else {
                sigset_t sigm;
+               tcp->u_rval = tcp->u_error = 0;
+               if (tcp->u_arg[0] == 0)
+                       return 0;
+               long_to_sigset(tcp->u_arg[1], &sigm);
+               tcp->auxstr = sprintsigmask("mask now ", &sigm, 0);
+               return RVAL_NONE | RVAL_STR;
+       }
+       return 0;
+#elif defined(CRISV10) || defined(CRISV32)
+       if (entering(tcp)) {
+               struct sigcontext sc;
+               long regs[PT_MAX+1];
+               tcp->u_arg[0] = 0;
+               if (ptrace(PTRACE_GETREGS, tcp->pid, NULL, (long)regs) < 0) {
+                       perror("sigreturn: PTRACE_GETREGS");
+                       return 0;
+               }
+               if (umove(tcp, regs[PT_USP], &sc) < 0)
+                       return 0;
+               tcp->u_arg[0] = 1;
+               tcp->u_arg[1] = sc.oldmask;
+       } else {
+               sigset_t sigm;
+               tcp->u_rval = tcp->u_error = 0;
+               if (tcp->u_arg[0] == 0)
+                       return 0;
                long_to_sigset(tcp->u_arg[1], &sigm);
-               tcp->u_rval = tcp->u_error = 0;
-               if(tcp->u_arg[0] == 0)
+               tcp->auxstr = sprintsigmask("mask now ", &sigm, 0);
+               return RVAL_NONE | RVAL_STR;
+       }
+       return 0;
+#elif defined(TILE)
+       if (entering(tcp)) {
+               struct ucontext uc;
+               long sp;
+
+               /* offset of ucontext in the kernel's sigframe structure */
+#              define SIGFRAME_UC_OFFSET C_ABI_SAVE_AREA_SIZE + sizeof(struct siginfo)
+               tcp->u_arg[0] = 0;
+               if (upeek(tcp, PTREGS_OFFSET_SP, &sp) < 0)
+                       return 0;
+               if (umove(tcp, sp + SIGFRAME_UC_OFFSET, &uc) < 0)
+                       return 0;
+               tcp->u_arg[0] = 1;
+               memcpy(tcp->u_arg + 1, &uc.uc_sigmask, NSIG / 8);
+       }
+       else {
+               sigset_t sigm;
+               tcp->u_rval = tcp->u_error = 0;
+               if (tcp->u_arg[0] == 0)
+                       return 0;
+               sigemptyset(&sigm);
+               memcpy(&sigm, tcp->u_arg + 1, NSIG / 8);
+               tcp->auxstr = sprintsigmask("mask now ", &sigm, 0);
+               return RVAL_NONE | RVAL_STR;
+       }
+       return 0;
+#elif defined(MICROBLAZE)
+       /* TODO: Verify that this is correct...  */
+       if (entering(tcp)) {
+               struct sigcontext sc;
+               long sp;
+               tcp->u_arg[0] = 0;
+               /* Read r1, the stack pointer.  */
+               if (upeek(tcp, 1 * 4, &sp) < 0)
+                       return 0;
+               if (umove(tcp, sp, &sc) < 0)
+                       return 0;
+               tcp->u_arg[0] = 1;
+               tcp->u_arg[1] = sc.oldmask;
+       } else {
+               sigset_t sigm;
+               tcp->u_rval = tcp->u_error = 0;
+               if (tcp->u_arg[0] == 0)
                        return 0;
+               long_to_sigset(tcp->u_arg[1], &sigm);
                tcp->auxstr = sprintsigmask("mask now ", &sigm, 0);
                return RVAL_NONE | RVAL_STR;
        }
@@ -1485,20 +1428,11 @@ struct tcb *tcp;
 #warning No sys_sigreturn() for this architecture
 #warning         (no problem, just a reminder :-)
        return 0;
-#endif /* MIPS */
-#endif /* LINUX_MIPSN32 || LINUX_MIPSN64 */
-#endif /* SPARC || SPARC64 */
-#endif /* ALPHA */
-#endif /* !M68K */
-#endif /* !POWERPC */
-#endif /* !IA64 */
-#endif /* !I386 */
-#endif /* S390 */
+#endif
 }
 
 int
-sys_siggetmask(tcp)
-struct tcb *tcp;
+sys_siggetmask(struct tcb *tcp)
 {
        if (exiting(tcp)) {
                sigset_t sigm;
@@ -1509,17 +1443,11 @@ struct tcb *tcp;
 }
 
 int
-sys_sigsuspend(tcp)
-struct tcb *tcp;
+sys_sigsuspend(struct tcb *tcp)
 {
        if (entering(tcp)) {
                sigset_t sigm;
                long_to_sigset(tcp->u_arg[2], &sigm);
-#if 0
-               /* first two are not really arguments, but print them anyway */
-               /* nevermind, they are an anachronism now, too bad... */
-               tprintf("%d, %#x, ", tcp->u_arg[0], tcp->u_arg[1]);
-#endif
                printsigmask(&sigm, 0);
        }
        return 0;
@@ -1530,8 +1458,7 @@ struct tcb *tcp;
 #if defined(SVR4) || defined(FREEBSD)
 
 int
-sys_sigsuspend(tcp)
-struct tcb *tcp;
+sys_sigsuspend(struct tcb *tcp)
 {
        sigset_t sigset;
 
@@ -1585,9 +1512,7 @@ static const struct xlat sigaltstack_flags[] = {
 
 #ifdef SVR4
 static void
-printcontext(tcp, ucp)
-struct tcb *tcp;
-ucontext_t *ucp;
+printcontext(struct tcb *tcp, ucontext_t *ucp)
 {
        tprintf("{");
        if (!abbrev(tcp)) {
@@ -1608,8 +1533,7 @@ ucontext_t *ucp;
 }
 
 int
-sys_getcontext(tcp)
-struct tcb *tcp;
+sys_getcontext(struct tcb *tcp)
 {
        ucontext_t uc;
 
@@ -1627,8 +1551,7 @@ struct tcb *tcp;
 }
 
 int
-sys_setcontext(tcp)
-struct tcb *tcp;
+sys_setcontext(struct tcb *tcp)
 {
        ucontext_t uc;
 
@@ -1654,9 +1577,7 @@ struct tcb *tcp;
 #if defined(LINUX) || defined(FREEBSD)
 
 static int
-print_stack_t(tcp, addr)
-struct tcb *tcp;
-unsigned long addr;
+print_stack_t(struct tcb *tcp, unsigned long addr)
 {
        stack_t ss;
        if (umove(tcp, addr, &ss) < 0)
@@ -1668,8 +1589,7 @@ unsigned long addr;
 }
 
 int
-sys_sigaltstack(tcp)
-       struct tcb *tcp;
+sys_sigaltstack(struct tcb *tcp)
 {
        if (entering(tcp)) {
                if (tcp->u_arg[0] == 0)
@@ -1691,8 +1611,7 @@ sys_sigaltstack(tcp)
 #ifdef HAVE_SIGACTION
 
 int
-sys_sigprocmask(tcp)
-struct tcb *tcp;
+sys_sigprocmask(struct tcb *tcp)
 {
 #ifdef ALPHA
        if (entering(tcp)) {
@@ -1731,19 +1650,23 @@ struct tcb *tcp;
 #endif /* HAVE_SIGACTION */
 
 int
-sys_kill(tcp)
-struct tcb *tcp;
+sys_kill(struct tcb *tcp)
 {
        if (entering(tcp)) {
-               tprintf("%ld, %s", tcp->u_arg[0], signame(tcp->u_arg[1]));
+               long pid = tcp->u_arg[0];
+#if SUPPORTED_PERSONALITIES > 1
+               /* Sign-extend a 32-bit value when that's what it is. */
+               if (personality_wordsize[current_personality] < sizeof pid)
+                       pid = (long) (int) pid;
+#endif
+               tprintf("%ld, %s", pid, signame(tcp->u_arg[1]));
        }
        return 0;
 }
 
 #if defined(FREEBSD) || defined(SUNOS4)
 int
-sys_killpg(tcp)
-struct tcb *tcp;
+sys_killpg(struct tcb *tcp)
 {
        return sys_kill(tcp);
 }
@@ -1751,8 +1674,7 @@ struct tcb *tcp;
 
 #ifdef LINUX
 int
-sys_tgkill(tcp)
-       struct tcb *tcp;
+sys_tgkill(struct tcb *tcp)
 {
        if (entering(tcp)) {
                tprintf("%ld, %ld, %s",
@@ -1763,8 +1685,7 @@ sys_tgkill(tcp)
 #endif
 
 int
-sys_sigpending(tcp)
-struct tcb *tcp;
+sys_sigpending(struct tcb *tcp)
 {
        sigset_t sigset;
 
@@ -1780,8 +1701,7 @@ struct tcb *tcp;
 }
 
 #ifdef SVR4
-int sys_sigwait(tcp)
-struct tcb *tcp;
+int sys_sigwait(struct tcb *tcp)
 {
        sigset_t sigset;
 
@@ -1803,9 +1723,8 @@ struct tcb *tcp;
 
 #ifdef LINUX
 
-       int
-sys_rt_sigprocmask(tcp)
-       struct tcb *tcp;
+int
+sys_rt_sigprocmask(struct tcb *tcp)
 {
        sigset_t sigset;
 
@@ -1824,7 +1743,6 @@ sys_rt_sigprocmask(tcp)
        }
        else {
                if (!tcp->u_arg[2])
-
                        tprintf("NULL");
                else if (syserror(tcp))
                        tprintf("%#lx", tcp->u_arg[2]);
@@ -1841,25 +1759,29 @@ sys_rt_sigprocmask(tcp)
 /* Structure describing the action to be taken when a signal arrives.  */
 struct new_sigaction
 {
-       union
-       {
-               __sighandler_t __sa_handler;
-               void (*__sa_sigaction) (int, siginfo_t *, void *);
-       }
-       __sigaction_handler;
+       __sighandler_t __sa_handler;
        unsigned long sa_flags;
        void (*sa_restorer) (void);
-       unsigned long int sa_mask[2];
+       /* Kernel treats sa_mask as an array of longs. */
+       unsigned long sa_mask[NSIG / sizeof(long) ? NSIG / sizeof(long) : 1];
+};
+/* Same for i386-on-x86_64 and similar cases */
+struct new_sigaction32
+{
+       uint32_t __sa_handler;
+       uint32_t sa_flags;
+       uint32_t sa_restorer;
+       uint32_t sa_mask[2 * (NSIG / sizeof(long) ? NSIG / sizeof(long) : 1)];
 };
 
 
-       int
-sys_rt_sigaction(tcp)
-       struct tcb *tcp;
+int
+sys_rt_sigaction(struct tcb *tcp)
 {
        struct new_sigaction sa;
        sigset_t sigset;
        long addr;
+       int r;
 
        if (entering(tcp)) {
                printsignal(tcp->u_arg[0]);
@@ -1867,46 +1789,86 @@ sys_rt_sigaction(tcp)
                addr = tcp->u_arg[1];
        } else
                addr = tcp->u_arg[2];
-       if (addr == 0)
+
+       if (addr == 0) {
                tprintf("NULL");
-       else if (!verbose(tcp))
+               goto after_sa;
+       }
+       if (!verbose(tcp)) {
                tprintf("%#lx", addr);
-       else if (umove(tcp, addr, &sa) < 0)
+               goto after_sa;
+       }
+#if SUPPORTED_PERSONALITIES > 1
+       if (personality_wordsize[current_personality] != sizeof(sa.sa_flags)
+        && personality_wordsize[current_personality] == 4
+       ) {
+               struct new_sigaction32 sa32;
+               r = umove(tcp, addr, &sa32);
+               if (r >= 0) {
+                       memset(&sa, 0, sizeof(sa));
+                       sa.__sa_handler = (void*)(unsigned long)sa32.__sa_handler;
+                       sa.sa_flags     = sa32.sa_flags;
+                       sa.sa_restorer  = (void*)(unsigned long)sa32.sa_restorer;
+                       /* Kernel treats sa_mask as an array of longs.
+                        * For 32-bit process, "long" is uint32_t, thus, for example,
+                        * 32th bit in sa_mask will end up as bit 0 in sa_mask[1].
+                        * But for (64-bit) kernel, 32th bit in sa_mask is
+                        * 32th bit in 0th (64-bit) long!
+                        * For little-endian, it's the same.
+                        * For big-endian, we swap 32-bit words.
+                        */
+                       sa.sa_mask[0] = sa32.sa_mask[0] + ((long)(sa32.sa_mask[1]) << 32);
+               }
+       } else
+#endif
+       {
+               r = umove(tcp, addr, &sa);
+       }
+       if (r < 0) {
                tprintf("{...}");
-       else {
-               switch ((long) sa.__sigaction_handler.__sa_handler) {
-                       case (long) SIG_ERR:
-                               tprintf("{SIG_ERR}");
-                               break;
-                       case (long) SIG_DFL:
-                               tprintf("{SIG_DFL}");
-                               break;
-                       case (long) SIG_IGN:
-                               tprintf("{SIG_IGN}");
-                               break;
-                       default:
-                               tprintf("{%#lx, ",
-                                               (long) sa.__sigaction_handler.__sa_handler);
-                               sigemptyset(&sigset);
+               goto after_sa;
+       }
+       /* Architectures using function pointers, like
+        * hppa, may need to manipulate the function pointer
+        * to compute the result of a comparison. However,
+        * the SA_HANDLER function pointer exists only in
+        * the address space of the traced process, and can't
+        * be manipulated by strace. In order to prevent the
+        * compiler from generating code to manipulate
+        * SA_HANDLER we cast the function pointers to long. */
+       if ((long)sa.__sa_handler == (long)SIG_ERR)
+               tprintf("{SIG_ERR, ");
+       else if ((long)sa.__sa_handler == (long)SIG_DFL)
+               tprintf("{SIG_DFL, ");
+       else if ((long)sa.__sa_handler == (long)SIG_IGN)
+               tprintf("{SIG_IGN, ");
+       else
+               tprintf("{%#lx, ", (long) sa.__sa_handler);
+       /* Questionable code below.
+        * Kernel won't handle sys_rt_sigaction
+        * with wrong sigset size (just returns EINVAL)
+        * therefore tcp->u_arg[3(4)] _must_ be NSIG / 8 here,
+        * and we always use smaller memcpy. */
+       sigemptyset(&sigset);
 #ifdef LINUXSPARC
-                               if (tcp->u_arg[4] <= sizeof(sigset))
-                                       memcpy(&sigset, &sa.sa_mask, tcp->u_arg[4]);
+       if (tcp->u_arg[4] <= sizeof(sigset))
+               memcpy(&sigset, &sa.sa_mask, tcp->u_arg[4]);
 #else
-                               if (tcp->u_arg[3] <= sizeof(sigset))
-                                       memcpy(&sigset, &sa.sa_mask, tcp->u_arg[3]);
+       if (tcp->u_arg[3] <= sizeof(sigset))
+               memcpy(&sigset, &sa.sa_mask, tcp->u_arg[3]);
 #endif
-                               else
-                                       memcpy(&sigset, &sa.sa_mask, sizeof(sigset));
-                               printsigmask(&sigset, 1);
-                               tprintf(", ");
-                               printflags(sigact_flags, sa.sa_flags, "SA_???");
+       else
+               memcpy(&sigset, &sa.sa_mask, sizeof(sigset));
+       printsigmask(&sigset, 1);
+       tprintf(", ");
+       printflags(sigact_flags, sa.sa_flags, "SA_???");
 #ifdef SA_RESTORER
-                               if (sa.sa_flags & SA_RESTORER)
-                                       tprintf(", %p", sa.sa_restorer);
+       if (sa.sa_flags & SA_RESTORER)
+               tprintf(", %p", sa.sa_restorer);
 #endif
-                               tprintf("}");
-               }
-       }
+       tprintf("}");
+
+ after_sa:
        if (entering(tcp))
                tprintf(", ");
        else
@@ -1915,14 +1877,13 @@ sys_rt_sigaction(tcp)
 #elif defined(ALPHA)
                tprintf(", %lu, %#lx", tcp->u_arg[3], tcp->u_arg[4]);
 #else
-               tprintf(", %lu", addr = tcp->u_arg[3]);
+               tprintf(", %lu", tcp->u_arg[3]);
 #endif
        return 0;
 }
 
-       int
-sys_rt_sigpending(tcp)
-       struct tcb *tcp;
+int
+sys_rt_sigpending(struct tcb *tcp)
 {
        sigset_t sigset;
 
@@ -1937,9 +1898,9 @@ sys_rt_sigpending(tcp)
        }
        return 0;
 }
-       int
-sys_rt_sigsuspend(tcp)
-       struct tcb *tcp;
+
+int
+sys_rt_sigsuspend(struct tcb *tcp)
 {
        if (entering(tcp)) {
                sigset_t sigm;
@@ -1950,9 +1911,9 @@ sys_rt_sigsuspend(tcp)
        }
        return 0;
 }
-       int
-sys_rt_sigqueueinfo(tcp)
-       struct tcb *tcp;
+
+int
+sys_rt_sigqueueinfo(struct tcb *tcp)
 {
        if (entering(tcp)) {
                siginfo_t si;
@@ -1962,13 +1923,12 @@ sys_rt_sigqueueinfo(tcp)
                if (umove(tcp, tcp->u_arg[2], &si) < 0)
                        tprintf("%#lx", tcp->u_arg[2]);
                else
-                       printsiginfo(&si, verbose (tcp));
+                       printsiginfo(&si, verbose(tcp));
        }
        return 0;
 }
 
-int sys_rt_sigtimedwait(tcp)
-       struct tcb *tcp;
+int sys_rt_sigtimedwait(struct tcb *tcp)
 {
        if (entering(tcp)) {
                sigset_t sigset;
@@ -1979,42 +1939,68 @@ int sys_rt_sigtimedwait(tcp)
                else
                        printsigmask(&sigset, 1);
                tprintf(", ");
+               /* This is the only "return" parameter, */
+               if (tcp->u_arg[1] != 0)
+                       return 0;
+               /* ... if it's NULL, can decode all on entry */
+               tprintf("NULL, ");
        }
-       else {
+       else if (tcp->u_arg[1] != 0) {
+               /* syscall exit, and u_arg[1] wasn't NULL */
                if (syserror(tcp))
-                       tprintf("%#lx", tcp->u_arg[0]);
+                       tprintf("%#lx, ", tcp->u_arg[1]);
                else {
                        siginfo_t si;
                        if (umove(tcp, tcp->u_arg[1], &si) < 0)
-                               tprintf("%#lx", tcp->u_arg[1]);
-                       else
-                               printsiginfo(&si, verbose (tcp));
-                       /* XXX For now */
-                       tprintf(", %#lx", tcp->u_arg[2]);
-                       tprintf(", %d", (int) tcp->u_arg[3]);
+                               tprintf("%#lx, ", tcp->u_arg[1]);
+                       else {
+                               printsiginfo(&si, verbose(tcp));
+                               tprintf(", ");
+                       }
                }
        }
+       else {
+               /* syscall exit, and u_arg[1] was NULL */
+               return 0;
+       }
+       print_timespec(tcp, tcp->u_arg[2]);
+       tprintf(", %d", (int) tcp->u_arg[3]);
        return 0;
 };
 
 int
-sys_restart_syscall(tcp)
-struct tcb *tcp;
+sys_restart_syscall(struct tcb *tcp)
 {
        if (entering(tcp))
                tprintf("<... resuming interrupted call ...>");
        return 0;
 }
 
-int
-sys_signalfd(tcp)
-struct tcb *tcp;
+static int
+do_signalfd(struct tcb *tcp, int flags_arg)
 {
        if (entering(tcp)) {
-               tprintf("%ld, ", tcp->u_arg[0]);
+               printfd(tcp, tcp->u_arg[0]);
+               tprintf(", ");
                print_sigset(tcp, tcp->u_arg[1], 1);
-               tprintf("%lu", tcp->u_arg[2]);
+               tprintf(", %lu", tcp->u_arg[2]);
+               if (flags_arg >= 0) {
+                       tprintf(", ");
+                       printflags(open_mode_flags, tcp->u_arg[flags_arg], "O_???");
+               }
        }
        return 0;
 }
+
+int
+sys_signalfd(struct tcb *tcp)
+{
+       return do_signalfd(tcp, -1);
+}
+
+int
+sys_signalfd4(struct tcb *tcp)
+{
+       return do_signalfd(tcp, 3);
+}
 #endif /* LINUX */