Re: CVS commit: src/sys/arch/i386/include

[Matthias Drochner <M.Drochner%fz-juelich.de@localhost>]

christos%zoulas.com@localhost said:
> No, si_code is reserved for MI stuff so we cannot use this. Read the
> new manpage for siginfo... 

So wouldn't the the appended patch make signals to sa programs work better?
(This is related to a test case I've sent to Nathan a while ago, where traps
with code=0 did confuse libpthread's signal delivery code.)

best regards
Matthias


Index: kern/kern_sig.c
===================================================================
RCS file: /cvsroot/src/sys/kern/kern_sig.c,v
retrieving revision 1.149
diff -u -p -r1.149 kern_sig.c
--- kern/kern_sig.c     2003/09/10 16:41:26     1.149
+++ kern/kern_sig.c     2003/09/10 18:28:18
@@ -1168,7 +1168,7 @@ kpsendsig(struct lwp *l, ksiginfo_t *ksi
                si = pool_get(&siginfo_pool, PR_WAITOK);
                si->_info = *ksi;
                le = li = NULL;
-               if (ksi->ksi_trap)
+               if (ksi->ksi_code > 0)
                        le = l;
                else
                        li = l;