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