From patchwork Fri Mar 1 15:29:39 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dan Carpenter X-Patchwork-Id: 208896 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:2097:b0:108:e6aa:91d0 with SMTP id gs23csp1158467dyb; Fri, 1 Mar 2024 07:33:24 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCW+QH6Z64SVsx9tqU0sBqbXeEwQ0PqP7jBdU6JUu/qSh4SYXh+p/fxcnOK+LoC/GNp46dyHly0eOcu/iLsnL7KX6kCl4g== X-Google-Smtp-Source: AGHT+IFcj7bM0QQoFFD903kLb9tHdw0mHlcDCH5OLLlwm5DP8JyuP65HKBKk1V7wHwModQ/qaeP2 X-Received: by 2002:a17:902:a384:b0:1dc:a5fd:6de9 with SMTP id x4-20020a170902a38400b001dca5fd6de9mr1972603pla.12.1709307204386; Fri, 01 Mar 2024 07:33:24 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1709307204; cv=pass; d=google.com; s=arc-20160816; b=oU2HZD/GABPUvhKLE7u0Unax6UJSVK/uFlm0uFPZbraSap/NDDYpxPHbvUvxEaBste zmv7bCdXb8xb2W6arYnl6anxnFynxFN5CyVDrLYLiQV+62Aw7RilDG/7mD1a4+q0XhFi 08PEVxjN/TGmpGyPxGinWQP5ZIxqR5EhT49hVRqS+YVcQRpAobwarpbP6N+uyA8Hh+zt UhEy75Qqc4Nm4xJKD3Rn3X+z4sEj0U13diCFlJNJM3wmhSDF0cncpL/RJ2RmykkMeDY/ d18T//OGBSzr373BSP8ql3jkgTN4vMPsEWvoAHZSVP1wlxtLXXtS3n+yF/Oy847GIo7C fAKg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:message-id:subject:cc:to:from :date:dkim-signature; bh=A9xdheLv/6a2bwYercXwXyeHsHuRfad0YKjyOjEInbQ=; fh=Hqu2fbcJEg2JzAkUipdwJB+GnU+uM9YGwMT6kk1YLRQ=; b=Nv22ljBZP4ooeajmQtnnsnHigg3Q6IAwzjfpoiZa5gZoEr19oUartznATFzAYTT609 TR3mjmx6yOhhMo+3V5pem4KBRvJQ8LvjGZ/8a8jEMR0UfWhHO0t0kW9zEkT/6DmprgYJ GGScYgUMFZZQZ4Oi0LxNzPl0RjyIahNWqMOK45j6eTfXbVZh81urmW08pWCeHN1hh7fC miEnNXOfFUvbCuLQZll8hqW6Ft4ZGwRZghmf5XDBkYdtEeZFDhU0NUYBqCBzDLMcyR4D ITC9qNPAc3cCWp39wdSFm1JONhxt451MD4LkAtpNkkI69Sd0G1WE3JFU7pyWBPDNSS9S y2Xg==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b="q0+ZC4T/"; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-88623-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-88623-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from sy.mirrors.kernel.org (sy.mirrors.kernel.org. [2604:1380:40f1:3f00::1]) by mx.google.com with ESMTPS id e5-20020a17090301c500b001dbc4d9d82csi3737565plh.479.2024.03.01.07.33.23 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 01 Mar 2024 07:33:24 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-88623-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b="q0+ZC4T/"; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-88623-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-88623-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sy.mirrors.kernel.org (Postfix) with ESMTPS id 670FEB23F76 for ; Fri, 1 Mar 2024 15:30:16 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 64F9B70AFA; Fri, 1 Mar 2024 15:29:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="q0+ZC4T/" Received: from mail-wm1-f53.google.com (mail-wm1-f53.google.com [209.85.128.53]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0721B6E5EB for ; Fri, 1 Mar 2024 15:29:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.53 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709306986; cv=none; b=QTuEyxuoBJAtsQZYFYkOV8ygvam6uLYhNpXblJqsMKtyzIcE3EMjMkTnh7JcevkDgBL0PZm4zk6ES3+YFvZS8DX+iE4YyYNmhnJ0fR4SEIRnN5ZLMYGJ42prXqvjz9MlnZ2ArQ+d39Vt0TkhZ32IXHvUvGHECnXDgr0q+o2gp20= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709306986; c=relaxed/simple; bh=5ZdVDMXs3mwTr4vWqpROwmQUzW0VeWSwZH0+3nB9a6A=; h=Date:From:To:Cc:Subject:Message-ID:MIME-Version:Content-Type: Content-Disposition:In-Reply-To; b=ZPhZ9ekPg27Q8G6pjsGLlvcY5RfwPflPSo9LiAI/675Sdbp5hnhrsMvZFNefhHddEDWSTiaZ1tdEEqrvRnuvZqDm79Vhg2SY+ts67pN12KNzWpAG2SbOo9Y/m5LCsLe7xGbFjHQdJKjVK7zsYHOfsWvrEHK9t7JXcv64GCob0jg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=q0+ZC4T/; arc=none smtp.client-ip=209.85.128.53 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Received: by mail-wm1-f53.google.com with SMTP id 5b1f17b1804b1-412c780464dso6095035e9.0 for ; Fri, 01 Mar 2024 07:29:44 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1709306983; x=1709911783; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:message-id:subject:cc :to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=A9xdheLv/6a2bwYercXwXyeHsHuRfad0YKjyOjEInbQ=; b=q0+ZC4T/71iLdg16IhHSC/synJmVczkJseZeCNfmPn1qDFKotgzxb8wnW183Hgk3EM wiskBJl1jY28QnFeOVYfSztid724P2+7NW7Dw7pLIhaui2DD19L6Y68AwNQHsS8R7S2B igNXHQHn85NOvTSfiLaJIbOkb0eHcIPRMms1EPB/P2NrKdXjec+4BWUZn3IXtFbCF33T JyrGF8MttGt4umpZDpJ51pqEx/VTTEUuMKFF3VssQ6GX/2jfRYI1Ee0fWoaS0lwPG8h+ y5wame0LMrmTTY2bAffdGpLVH2QhiiEfkF+4HFfn+vdc2V47Zp7ArEf0WazNRuxRsaIz Znaw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1709306983; x=1709911783; h=in-reply-to:content-disposition:mime-version:message-id:subject:cc :to:from:date:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=A9xdheLv/6a2bwYercXwXyeHsHuRfad0YKjyOjEInbQ=; b=KvdjIxJ/JmqopI6Lj5ihxcyFrGIGTVIXNqGI++I2BFDdeoX1LUqKPbUSL22C+XXfhS xIdw9HDS4G160JqkaSMhN9ciSp3kYWOVUBtUTrILq4lFZL++WeVAqRWoHBbJAcd4QcKP GTWGKfz4hRmFUBEwramuK5cGOSHv9GC1AVguY8yyPDMMlkm7Tpeo3hjvraiJ/1AC7msb WvoAPIPvQhPBiF/vutoN19NBSCG/g08vyW/5gO02Rae38YY18DAEaMvrMNelhcuFSDri sNqlGVQdveW9wE6IT9CUmJYQT3qJxI08Y/uHRdZByQCtidwtZzvt+8VXuUIkgjw2aF7d aFig== X-Forwarded-Encrypted: i=1; AJvYcCXn6gOXi1d37mzB5ELroPq/X1Nter1Iq3yjGt+vzo4U8HrDX1hNL9eVgJV2Wj8M+wYbIbyA0KpXNxv87Heoo32Jgs/cYHoslPskjdca X-Gm-Message-State: AOJu0Yw8IDVuyS0gJ89KRxPJ2XSn/vM1epzcO98M37tn1jQ+WnOm8Kw7 6X/X9F4bSvxNyzTndiYddjkVdA9Y4fBybAFJKj8Xo55Tj8H1u9z/U7cvSwoc5pM= X-Received: by 2002:a05:600c:4f83:b0:412:268f:1fa4 with SMTP id n3-20020a05600c4f8300b00412268f1fa4mr1731633wmq.1.1709306983141; Fri, 01 Mar 2024 07:29:43 -0800 (PST) Received: from localhost ([102.222.70.76]) by smtp.gmail.com with ESMTPSA id jd20-20020a05600c68d400b004128fa77216sm8827201wmb.1.2024.03.01.07.29.42 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 01 Mar 2024 07:29:42 -0800 (PST) Date: Fri, 1 Mar 2024 18:29:39 +0300 From: Dan Carpenter To: Dylan Yudaken Cc: Jens Axboe , Pavel Begunkov , io-uring@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: [PATCH 1/2] io_uring/net: fix overflow check in io_recvmsg_mshot_prep() Message-ID: <138bd2e2-ede8-4bcc-aa7b-f3d9de167a37@moroto.mountain> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <7f5d7887-f76e-4e68-98c2-894bfedbf292@moroto.mountain> X-Mailer: git-send-email haha only kidding X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1792338511026212556 X-GMAIL-MSGID: 1792338511026212556 The "controllen" variable is type size_t (unsigned long). Casting it to int could lead to an integer underflow. The check_add_overflow() function considers the type of the destination which is type int. If we add two positive values and the result cannot fit in an integer then that's counted as an overflow. However, if we cast "controllen" to an int and it turns negative, then negative values *can* fit into an int type so there is no overflow. Good: 100 + (unsigned long)-4 = 96 <-- overflow Bad: 100 + (int)-4 = 96 <-- no overflow I deleted the cast of the sizeof() as well. That's not a bug but the cast is unnecessary. Fixes: 9b0fc3c054ff ("io_uring: fix types in io_recvmsg_multishot_overflow") Signed-off-by: Dan Carpenter --- io_uring/net.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/io_uring/net.c b/io_uring/net.c index 926d1fb0335d..da257bf429d5 100644 --- a/io_uring/net.c +++ b/io_uring/net.c @@ -559,10 +559,10 @@ static int io_recvmsg_mshot_prep(struct io_kiocb *req, if (unlikely(namelen < 0)) return -EOVERFLOW; - if (check_add_overflow((int)sizeof(struct io_uring_recvmsg_out), + if (check_add_overflow(sizeof(struct io_uring_recvmsg_out), namelen, &hdr)) return -EOVERFLOW; - if (check_add_overflow(hdr, (int)controllen, &hdr)) + if (check_add_overflow(hdr, controllen, &hdr)) return -EOVERFLOW; iomsg->namelen = namelen; From patchwork Fri Mar 1 15:29:52 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dan Carpenter X-Patchwork-Id: 208897 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:2097:b0:108:e6aa:91d0 with SMTP id gs23csp1158829dyb; Fri, 1 Mar 2024 07:33:56 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCWqgPL5JcaAimWykq4Ru1HB+1/3q4Nuc/7gdk9abLnLKeMdS/hMZuLz97sESFiaqjrHqiX9YVxSJbwVknaW10Xh0TRUTw== X-Google-Smtp-Source: AGHT+IEM3alXZcHRbDVqziJI89cc0EwWaKN/vsp1XZvGzOUiymDzNDrGRnbED6kuEaDi9Rd1YE2P X-Received: by 2002:a05:6a21:7884:b0:1a1:431c:6696 with SMTP id bf4-20020a056a21788400b001a1431c6696mr314318pzc.49.1709307236005; Fri, 01 Mar 2024 07:33:56 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1709307235; cv=pass; d=google.com; s=arc-20160816; b=Yx5tLy9PFTVEwUMQIyJmNmVcPON/q9mk1uBbVjfD3+gX74vrzE6rHwMoEZPT8aCCs7 YZxQmzlMl8aex4i6MRqsttwBT1ChiYdDXUz8I0ordaMwquJr6MsGu8RtMQUVjCZEcDUX IvyDoh/nCfEPZCopYvc+AywA0I2KRIQMxB3pDAJyco61VDgoIqPmihZenwReliiNl+iZ W/IAB8gdIWyHTqVeH/ECCS5M4AQsuZHn97sRWjB6nDo6FAIKKN/98F9f8H2iQ1wL36GD DFQnSPt36C/tvsAYXsWp6mN4zONKmHPrhH/LGOn4eWCbRoO0lZTcKByozofLCGw4Xgdw mE9g== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:message-id:subject:cc:to:from :date:dkim-signature; bh=LwF1vEHHtNcLEL22JU7CbamIfjcY0v8w9f8KAiAGRpo=; fh=AP8lhwbJVt7oFeioIq/mx+lHs8smlfKz5PvYOsvDOSk=; b=LOOcPY7ycaE72ZBoiBZoxQQx84XjvQD7r1RkbAvsTt68oeZmPkRQOOSGajSOMv9plq jLqZj6Gf3Fp+U3UMoobAmHvnqcXudHN/eUWABq7t0wMjUysyCj646Ir+aTb2/UQdMBcT x8Gt+7TB+0C3tetd/i7slvFUVlUORVupq37lFl+Y1WsjS9t4y8EVwbWA9QwCM7X9pr+o YFfNClki+vREn0nq+e7u3tZ8hGndGAE9vfi8/QrcZ4W/KHHeLGTrZvCXUb2fC4rodPLm UcWhsHZn4rLqx48xLMuY8+2T9q5mhIp6OyS66DalOTd8mdfo+OnVNtMTwJd5DzO58lPw FoQQ==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=iN+Gj8BL; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-88624-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) smtp.mailfrom="linux-kernel+bounces-88624-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from sy.mirrors.kernel.org (sy.mirrors.kernel.org. [147.75.48.161]) by mx.google.com with ESMTPS id i17-20020a63cd11000000b005dc958db2fcsi3601274pgg.363.2024.03.01.07.33.55 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 01 Mar 2024 07:33:55 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-88624-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=iN+Gj8BL; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-88624-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) smtp.mailfrom="linux-kernel+bounces-88624-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sy.mirrors.kernel.org (Postfix) with ESMTPS id DF63EB24485 for ; Fri, 1 Mar 2024 15:30:35 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 86B7E70044; Fri, 1 Mar 2024 15:30:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="iN+Gj8BL" Received: from mail-lj1-f175.google.com (mail-lj1-f175.google.com [209.85.208.175]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0C4216F53D for ; Fri, 1 Mar 2024 15:29:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.175 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709306999; cv=none; b=hWPqoz7/6N86IMsJYdKpUpTxjCKZGikPLBh5PjiGmaxCC10SaNPpaWFqs34mln5FPbDMDBAMQJ7JLvW9lBz9e87kUlhHqPIQ756O8HqYLv0vmiwBf9URfNvi41Ech/vwMtUKO8+MfoejjkjGsJvhYVZXHlqhkZbf1kJcIiQiVZ4= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709306999; c=relaxed/simple; bh=zh1tMmI9C8esm9t8S5ODtHnOIgNunq7h2YrtfEFvx/Y=; h=Date:From:To:Cc:Subject:Message-ID:MIME-Version:Content-Type: Content-Disposition:In-Reply-To; b=jau09z1RQL9UgFA52xxbzLADu2wb3PJgSSC8CLSO5yTKtqQ2L4/ItpueXhG/o4roaxIyXL5Yx1bfhPdWo7DEEsoui/yEzjnhD3psZkaEEJcAfUVyKtUX/GlDq+F8CDG5Z6++2PhOTp4cL+QHz+u7gUAwykoM5SXmRJhdsjQ/lZM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=iN+Gj8BL; arc=none smtp.client-ip=209.85.208.175 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Received: by mail-lj1-f175.google.com with SMTP id 38308e7fff4ca-2d275e63590so29904021fa.2 for ; Fri, 01 Mar 2024 07:29:57 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1709306996; x=1709911796; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:message-id:subject:cc :to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=LwF1vEHHtNcLEL22JU7CbamIfjcY0v8w9f8KAiAGRpo=; b=iN+Gj8BLG7HaTyiBwK6YrhMrrIGiUH1KBbRM+WYLGIStLQUsdJiiDT8mAnZhyYDe0i c9M/sZSyJC3TxXEIAeRBVuAiHTKg6msl4k8SgObymvgGho+tdkul7JSAZm/ZYzpidVMj bFm3yJByorJR58EPp23wAxNsFc+DU7GFkuwsCqsUh+wbHJGPOINsql66cjrGOomlD5fw FSgwzISi1HG0hbsWJ2v+R2XowxCCzXND8GW3lWvgMt7eAH5bjqfc1XQXD0f37mVJpHIB 1ktYZPekGQZubSeNJiS8/IESaCq6PCGWpPnL9Ib2y9V6zRIToV1fOpdDRSbUsDMLAjY/ HVFg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1709306996; x=1709911796; h=in-reply-to:content-disposition:mime-version:message-id:subject:cc :to:from:date:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=LwF1vEHHtNcLEL22JU7CbamIfjcY0v8w9f8KAiAGRpo=; b=Bj4ww0/uY6aNPA9BOFPc/2sBPJFrDpmpc9EmnthBvpz7+2IC4FGXwQkiNo35m7z1t2 /W6yojUtTe/LXTJ1y/U9l7S6jED4H291fgS/aNaLtcFih5T4LUjRJHWTutG9njiLvHe5 h+pE+c+tYDrfuh3VdBk78JnqrbLi3++MvmfoPoTuE7JHIMTXjyC4JgBmJUXKBJHJ0VJZ 8LkCcMJViTzBAjlXFnsPL6d56Df41wb9EnnajoOrNHo+7z23MnEGk6IfJD9PTom6N8Sd +KkzIBN1GQeolGs4Bdr0XOTzhkpuAs2UJohGWrNmWrYBdgQP7G96pY2/HpVt7gVJ94yr aGag== X-Forwarded-Encrypted: i=1; AJvYcCVx4qgCgTP4LvcakLqYo3WOFMnjAk+NZ9JMNPztgy+w4h6F9UfO27EXH3MgIxChTP1oJnSHrOEPInJ07zIYTA/ZThp9gMTI3kSPbu/U X-Gm-Message-State: AOJu0YxEIlHGJCwwueqeYaFR6ZYnt2QQ0/5VDUIiOnc5xkK4J4f7mJmA uI6IZXFF7ZF5D8pciMK+M+ytvmMqe4A55paB9czt+9WpTCqXpeNjJvdD1rvs05c= X-Received: by 2002:a2e:2404:0:b0:2d2:935a:7a3d with SMTP id k4-20020a2e2404000000b002d2935a7a3dmr1312182ljk.34.1709306996297; Fri, 01 Mar 2024 07:29:56 -0800 (PST) Received: from localhost ([102.222.70.76]) by smtp.gmail.com with ESMTPSA id g8-20020a05600c310800b00412b0ef22basm5835531wmo.10.2024.03.01.07.29.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 01 Mar 2024 07:29:56 -0800 (PST) Date: Fri, 1 Mar 2024 18:29:52 +0300 From: Dan Carpenter To: Jens Axboe Cc: Pavel Begunkov , io-uring@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: [PATCH 2/2] io_uring/net: remove unnecessary check Message-ID: <3d17a814-2300-4902-8b2c-2a73c0e9bfc4@moroto.mountain> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <7f5d7887-f76e-4e68-98c2-894bfedbf292@moroto.mountain> X-Mailer: git-send-email haha only kidding X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1792338544201603532 X-GMAIL-MSGID: 1792338544201603532 "namelen" is type size_t so it can't be negative. Signed-off-by: Dan Carpenter --- io_uring/net.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/io_uring/net.c b/io_uring/net.c index da257bf429d5..04a7426c80d2 100644 --- a/io_uring/net.c +++ b/io_uring/net.c @@ -557,8 +557,6 @@ static int io_recvmsg_mshot_prep(struct io_kiocb *req, (REQ_F_APOLL_MULTISHOT|REQ_F_BUFFER_SELECT)) { int hdr; - if (unlikely(namelen < 0)) - return -EOVERFLOW; if (check_add_overflow(sizeof(struct io_uring_recvmsg_out), namelen, &hdr)) return -EOVERFLOW;