Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp2536663imu; Sat, 10 Nov 2018 17:34:30 -0800 (PST) X-Google-Smtp-Source: AJdET5deLjAciTWFYUhdJUeyBM/KeF5HIy7HJbUN+mwQM6hALQjyaormCgiUpkDjiFMh6jQ0JTts X-Received: by 2002:a62:4816:: with SMTP id v22-v6mr14615131pfa.233.1541900070655; Sat, 10 Nov 2018 17:34:30 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1541900070; cv=none; d=google.com; s=arc-20160816; b=Cxu3eyOLukiOhDGw5ZNFfUIR2teJTX/MksNHgV+zJWMhLT496RqQvO2+uISm5VFPLd yQAVkHKRlUd/yCzwiqcdy/EiZazSY7qy26F4Vrxj6HTz6FlOajsRWG2SkjtoAkEl2RdV PM2Y93u64eU2BMlCIjY+Lj9p9bMJWSQTAHhqMX/FelR8nDV9DM4LzMk85kW7mR9ygzFB KmIEmCn8+0efp0sDDOv4x3PE2+qfCiN5oma9fFwndOA4F2ziiHxr///z2N53CrXPOTIv H72kZ4nAoRxTPt7jqtDcdgxM0WcthqmfZ6Qsdzi18sTuCC95I9fJ69WA8+yejYDtm90L aRiA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:user-agent:references :in-reply-to:date:cc:to:from:subject:message-id; bh=x9bsedyUxmTrDBSz/AipHNxEMCf9EEJcRIfMF/+vB9w=; b=mkcPcLcCAnVUZGFrH5AAEsIrBk2kKdgdTUFDliD3CTA56mjCRtwsnw6p2IVcctpsmw PbnI2jJLx39Ipb/k8LA2MKEEAZJIobqSIhAUqt1/fMZySa4cGh6zVTy1pOWyoVhB91KS +zMr28d98HrF32WL2fYrrZT71cuFnBNV8THGQ+4bzRA39fcZGxN5A8037RKXpb8aDh5B 3MTD/jIhGRnC208vDtYh3nZ1W8Z8AE3OMamkfOcf+Co+CXlg+zak14oYv+SOKez8Ok2p D2hYHGPNIjAjO2DAVDNyDNljQV5km0AQqojaHLMJ+qL+QZEJ55L3JfMgFi8j+wbVXP0x FulQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id j6-v6si11638714pgb.62.2018.11.10.17.34.15; Sat, 10 Nov 2018 17:34:30 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727200AbeKKLUx (ORCPT + 99 others); Sun, 11 Nov 2018 06:20:53 -0500 Received: from shadbolt.e.decadent.org.uk ([88.96.1.126]:42884 "EHLO shadbolt.e.decadent.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726648AbeKKLUx (ORCPT ); Sun, 11 Nov 2018 06:20:53 -0500 Received: from [192.168.4.242] (helo=deadeye) by shadbolt.decadent.org.uk with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1gLedB-0001Rn-E5; Sun, 11 Nov 2018 01:33:41 +0000 Received: from ben by deadeye with local (Exim 4.91) (envelope-from ) id 1gLedB-00042V-5i; Sun, 11 Nov 2018 01:33:41 +0000 Message-ID: Subject: Re: [PATCH 3.16] staging: usbip stub_rx fix static checker warning on unnecessary checks From: Ben Hutchings To: Shuah Khan , valentina.manea.m@gmail.com, shuah@kernel.org, gregkh@linuxfoundation.org Cc: devel@driverdev.osuosl.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org, bwh@kernel.org, dan.carpenter@oracle.com Date: Sun, 11 Nov 2018 01:33:40 +0000 In-Reply-To: <20180226211241.19335-1-shuahkh@osg.samsung.com> References: <20180226211241.19335-1-shuahkh@osg.samsung.com> Content-Type: multipart/signed; micalg="pgp-sha512"; protocol="application/pgp-signature"; boundary="=-H7xMBZSXf9dND3GmbY2P" User-Agent: Evolution 3.30.1-1 Mime-Version: 1.0 X-SA-Exim-Connect-IP: 192.168.4.242 X-SA-Exim-Mail-From: ben@decadent.org.uk X-SA-Exim-Scanned: No (on shadbolt.decadent.org.uk); SAEximRunCond expanded to false Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --=-H7xMBZSXf9dND3GmbY2P Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Mon, 2018-02-26 at 14:12 -0700, Shuah Khan wrote: > Upstream commit 10c901209306 > ("usbip: stub_rx: fix static checker warning on unnecessary checks") >=20 > Back-port fix for static checker warning on unnecessary checks >=20 > smatch warnings: > drivers/staging/usbip/stub_rx.c:360 get_pipe() warn: impossible > condition '(pdu->u.cmd_submit.transfer_buffer_length > ((~0 >> 1))) =3D> > (s32min-s32max > s32max)' > drivers/staging/usbip/stub_rx.c:501 stub_recv_cmd_submit() warn: always > true condition '(pdu->u.cmd_submit.transfer_buffer_length <=3D ((~0 >> > 1))) =3D> (s32min-s32max <=3D s32max)' I've finally queued this up, thanks. Ben. > Reported-by: Dan Carpenter > Signed-off-by: Shuah Khan > --- > drivers/staging/usbip/stub_rx.c | 11 +---------- > 1 file changed, 1 insertion(+), 10 deletions(-) >=20 > diff --git a/drivers/staging/usbip/stub_rx.c b/drivers/staging/usbip/stub= _rx.c > index 35f59747122a..d8544ab9577b 100644 > --- a/drivers/staging/usbip/stub_rx.c > +++ b/drivers/staging/usbip/stub_rx.c > @@ -356,14 +356,6 @@ static int get_pipe(struct stub_device *sdev, struct= usbip_header *pdu) > =20 > epd =3D &ep->desc; > =20 > - /* validate transfer_buffer_length */ > - if (pdu->u.cmd_submit.transfer_buffer_length > INT_MAX) { > - dev_err(&sdev->udev->dev, > - "CMD_SUBMIT: -EMSGSIZE transfer_buffer_length %d\n", > - pdu->u.cmd_submit.transfer_buffer_length); > - return -1; > - } > - > if (usb_endpoint_xfer_control(epd)) { > if (dir =3D=3D USBIP_DIR_OUT) > return usb_sndctrlpipe(udev, epnum); > @@ -497,8 +489,7 @@ static void stub_recv_cmd_submit(struct stub_device *= sdev, > } > =20 > /* allocate urb transfer buffer, if needed */ > - if (pdu->u.cmd_submit.transfer_buffer_length > 0 && > - pdu->u.cmd_submit.transfer_buffer_length <=3D INT_MAX) { > + if (pdu->u.cmd_submit.transfer_buffer_length > 0) { > priv->urb->transfer_buffer =3D > kzalloc(pdu->u.cmd_submit.transfer_buffer_length, > GFP_KERNEL); --=20 Ben Hutchings Reality is just a crutch for people who can't handle science fiction. --=-H7xMBZSXf9dND3GmbY2P Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEErCspvTSmr92z9o8157/I7JWGEQkFAlvnhvQACgkQ57/I7JWG EQm+Vg/9GP1OCB9jSRsg30KsTTaPU14ZjdUfFdgpmRBF+vraruTA0PEj6N5uIXn+ 1PHYqx4keMY0SEF7j1JUiDk9Pow0YnwmoUt9etHSbS92sbMscluzFZ9b6uTEaRL3 MVClv5V46xnduSTgMNYEOh2+HiJI5ViEe0thtqM+bgUv8g/p/AUm5tYOd9aKT8o1 f2aVvi1Kk3ZOXI32Hk5/CS5fjowcCezL5/232UlyMz9MOvlz2UbwxELBB4qtJsYR v1/MyiC19JpbFEQSWteeFUj8ArYlWzPe0k8KlMpBiMfhk7giHL3QDJcLG6iF+GYy Yd8bVSAfdPc7GIW989tW37dsNZf6ud8CrVeRHEBJIooSI0jfjTR8Du29a5zYRh1y rYSaVzLanUv/C4L9FI3UClUE04aa7LE14m1WUNs1c0MNRPAi/ei/kQ0QqZtTlTXR Tpc58paPkio3RvQISg60mP6H9ePq/zapssQ61TDqNqeYlVu/ENJBpyUhY8tCJvbS f1jvesIsypv9duWA8UZaqH5tZfftaEiVq7G2Jb4cz6Yb6sW56FOf/peDsw8m2I9z w3Xjo22FAbOdHOp1/qxeNA1SSpW1V85N1WXj5Kirt7rY43uFcJxP3nLEOts3eihi QWgeBuFft9oDQB38971g6dMcHlMRZfdUXiOX6T6cieQXtrYa4xM= =hIb7 -----END PGP SIGNATURE----- --=-H7xMBZSXf9dND3GmbY2P--