truss bug + PATCH, pls review

Scott Long scottl at samsco.org
Thu Mar 3 14:34:35 GMT 2005


Alfred Perlstein wrote:
> Can someone review this?  I think 'u' is incorrectly
> added to instead of assigned to.  This causes the initial
> calculation to be garage based and screws up displaying
> poll information.
> 
> I'd like this to be MFC'd before 5.4 if possible.
> 
> Index: syscalls.c
> ===================================================================
> RCS file: /home/ncvs/src/usr.bin/truss/syscalls.c,v
> retrieving revision 1.45
> diff -u -r1.45 syscalls.c
> --- syscalls.c	5 Sep 2004 05:27:30 -0000	1.45
> +++ syscalls.c	2 Mar 2005 21:19:13 -0000
> @@ -414,7 +414,7 @@
>  	(POLLIN | POLLPRI | POLLOUT | POLLERR | POLLHUP | POLLNVAL | \
>  	 POLLRDNORM |POLLRDBAND | POLLWRBAND | POLLINIGNEOF) 
>  
> -	  u += snprintf(tmp + used, per_fd,
> +	  u = snprintf(tmp + used, per_fd,
>  	    "%s%d 0x%hx%s%s%s%s%s%s%s%s%s ",
>  	    i > 0 ? " " : "",
>  	    pfd[i].fd,
> 
> 

I agree with your analysis.

Scott


More information about the freebsd-hackers mailing list