HI Alan, Thanks for your code review. Rex > -----邮件原件----- > 发件人: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> > 发送时间: 2024年11月8日 22:09 > 收件人: Rex Nie <rex.nie@xxxxxxxxxxxxxxx> > 抄送: gregkh@xxxxxxxxxxxxxxxxxxx; linux-usb@xxxxxxxxxxxxxxx; > linux-kernel@xxxxxxxxxxxxxxx; Angus Chen <angus.chen@xxxxxxxxxxxxxxx>; > stable@xxxxxxxxxxxxxxx > 主题: Re: [PATCH] USB: core: remove dead code in do_proc_bulk() > > External Mail: This email originated from OUTSIDE of the organization! > Do not click links, open attachments or provide ANY information unless you > recognize the sender and know the content is safe. > > > On Fri, Nov 08, 2024 at 05:42:55PM +0800, Rex Nie wrote: > > Since len1 is unsigned int, len1 < 0 always false. Remove it keep code > > simple. > > > > Signed-off-by: Rex Nie <rex.nie@xxxxxxxxxxxxxxx> > > Acked-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> > > > --- > > drivers/usb/core/devio.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/usb/core/devio.c b/drivers/usb/core/devio.c index > > 3beb6a862e80..712e290bab04 100644 > > --- a/drivers/usb/core/devio.c > > +++ b/drivers/usb/core/devio.c > > @@ -1295,7 +1295,7 @@ static int do_proc_bulk(struct usb_dev_state *ps, > > return ret; > > > > len1 = bulk->len; > > - if (len1 < 0 || len1 >= (INT_MAX - sizeof(struct urb))) > > + if (len1 >= (INT_MAX - sizeof(struct urb))) > > return -EINVAL; > > > > if (bulk->ep & USB_DIR_IN) > > -- > > 2.17.1 > > > >
