Re: [PATCH]: if (cond); foo() in firewire

From: Adrian Chadd <adrian_at_freebsd.org>
Date: Mon, 22 Jun 2009 12:04:49 +0800
2009/6/21 Roman Divacky <rdivacky_at_freebsd.org>:
> 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)
> _at__at_ -443,7 +443,7 _at__at_
>        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" ;)

Is clang also evaluating all subsequent execution paths to tell you
what the change in program flow is? :)

I hate to be the harbinger of evilness, but I'd at least attempt a
cursory glance at the code to make sure subsequent code is doing the
right thing. (It certainly looks like a vanilla userland transfer!)

2c,


Adrian
Received on Mon Jun 22 2009 - 02:25:06 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:39:50 UTC