On Wed, Nov 20, 2019 at 12:07:10PM +0900, Nobuhiro Iwamatsu wrote: > From: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > > commit a56dcc6b455830776899ce3686735f1172e12243 upstream. > > This code is supposed to test for negative error codes and partial > reads, but because sizeof() is size_t (unsigned) type then negative > error codes are type promoted to high positive values and the condition > doesn't work as expected. > > Fixes: 332f989a3b00 ("CDC-NCM: handle incomplete transfer of MTU") > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > Signed-off-by: David S. Miller <davem@xxxxxxxxxxxxx> > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@xxxxxxxxxxxxx> > --- > drivers/net/usb/cdc_ncm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Now queud up, thanks! greg k-h