[PATCH]: if (cond); foo() in firewire
Jung-uk Kim
jkim at FreeBSD.org
Tue Jun 23 17:26:52 UTC 2009
On Sunday 21 June 2009 04:20 am, Roman Divacky wrote:
> hi
>
> is this patch correct? may I commit it?
>
> Index: ../../../dev/firewire/fwdev.c
> ===================================================================
> --- ../../../dev/firewire/fwdev.c (revision 194573)
> +++ ../../../dev/firewire/fwdev.c (working copy)
> @@ -443,7 +443,7 @@
> xfer->send.pay_len = uio->uio_resid;
> if (uio->uio_resid > 0) {
> if ((err = uiomove((caddr_t)&xfer->send.payload[0],
> - uio->uio_resid, uio)));
> + uio->uio_resid, uio)))
> goto out;
> }
>
>
> another bug found by the "useless warnings in clang" ;)
Just for the record, it seems this bug was fixed in NetBSD long ago:
http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/dev/ieee1394/fwdev.c.diff?r1=1.3&r2=1.4
I hope it boosts your confidence level a bit. :-)
Jung-uk Kim
More information about the freebsd-current
mailing list