cvs commit: src/sys/compat/svr4 svr4_signal.c svr4_signal.h
svr4_sysvec.c
Sam Leffler
sam at errno.com
Fri Feb 25 19:38:19 GMT 2005
Sam Leffler wrote:
> sam 2005-02-25 19:34:11 UTC
>
> FreeBSD src repository
>
> Modified files:
> sys/compat/svr4 svr4_signal.c svr4_signal.h svr4_sysvec.c
> Log:
> fixup signal mapping:
> o change the mapping arrays to have a zero offset rather than base 1;
> this eliminates lots of signo adjustments and brings the code
> back inline with the original netbsd code
> o purge use of SVR4_SIGTBLZ; SVR4_NSIG is the only definition for
> how big a mapping array is
> o change the mapping loops to explicitly ignore signal 0
> o purge some bogus code from bsd_to_svr4_sigset
> o adjust svr4_sysentvec to deal with the mapping table change
>
> Enticed into fixing by: Coverity Prevent analysis tool
> Glanced at by: marcel, jhb
Note that I do not have an svr4 binaries to run to test these changes.
I wrote some test applications to verify the signal mapping logic but if
someone can vet these changes with real use I'd appreciate it.
FWIW all this came about because the Coverity tool pointed out an
invalid array index that led me into this tarpit.
Sam
More information about the cvs-src
mailing list