Re: ehci-dbg.c:656 fill_periodic_buffer() warn: variable dereferenced before check 'p.qh' (see line 599)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 14 Sep 2012, Fengguang Wu wrote:

> Hi Richard,
> 
> FYI, there are new smatch warnings show up in
> 
> commit 3807e26d69b9ad3864fe03224ebebc9610d5802e
> Author:     Alek Du <alek.du@xxxxxxxxx>
> AuthorDate: Tue Jul 14 07:23:29 2009 +0800
> 
> drivers/usb/chipidea/../host/ehci-dbg.c:656 fill_periodic_buffer() warn: variable dereferenced before check 'p.qh' (see line 599)
> 
> vim +656 drivers/usb/chipidea/../host/ehci-dbg.c
>    596 
>    597                         switch (hc32_to_cpu(ehci, tag)) {
>    598                         case Q_TYPE_QH:
>    599                                 hw = p.qh->hw;
> ...
>  > 656					if (p.qh) {
>    657						tag = Q_NEXT_TYPE(ehci, hw->hw_next);
>    658						p = p.qh->qh_next;
>    659					}
>    660					break;

smatch is right; the test on line 656 isn't needed.  I'll take it out.

Thank you.

Alan Stern

--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux