From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f52.google.com (mail-wm1-f52.google.com [209.85.128.52]) (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 1548D3FB7D3 for ; Tue, 28 Apr 2026 13:52:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777384368; cv=none; b=WHT2r9+9dGK6mOMFCt8tLh1M0o2AZHXBtAEJ/u08uQMuHREE92JEFeUj2cQcPq5MRP9o8/lIC3uQKDP0GwUc8xAkBPOf+O4F2SjifZqhU8E7hgm2oGDsh3Yxk8/S4JcYGeOkkm49uEcrqrZQLv0JTMVzn/uKqZCre59alU1NVFw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777384368; c=relaxed/simple; bh=4Z6Hmsviejwbkx6SPgDRDj4FO2D2h8bGMTAA7pPjwAk=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=nCYB83DiHNsLqExdLY5W0zGNHGEGoYfagi/6c33ldhIJNMxUfJ+qWotM0JTNeQxcq05YeCLeI9UMK8U179+vZw/RqMZIvpEGb9dKlIn4kxCz1eNC+OAnedSqyfAgnGMlg/H66UdFYYL07J5q2vySq6oqDuHVFv4URmuxjAe/ES0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=H96EuF95; arc=none smtp.client-ip=209.85.128.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="H96EuF95" Received: by mail-wm1-f52.google.com with SMTP id 5b1f17b1804b1-48896199cbaso113091475e9.1 for ; Tue, 28 Apr 2026 06:52:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1777384365; x=1777989165; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=TvN+aQrEQG1INNwJXWKA667EIqAzFBBNzUYu75gRDbM=; b=H96EuF95cSsrXFUd3oQ05BHsRrWViJ8v/XipbGNi+4JQwPw7YgMAEpQfbuZa92x6Vr U6M61WCI0QyW4efOLJI0Lq5+aDByQTLtVnssYO0nTeNZJAUcz+8Mn3EWzLs7+Y1bQIou 6Z3hyovUF/1/qUFMGzNa2JGAe2HqITs9Y4LJHqxNTISFt7nhjxXi1yHESEX9XAZCL/Vd kLyWJwvzo33vvX3+Re9Aio7IEsO+vkpEEgZnmiVAkvUesAmNnPtp1YRcYndrNQ4reiQq vXjAVTPZFQNHUc6ng903FVgqiDpC1/4SVLUW9zw5W6sIgS4V4chwkeNoGEfzkmervdV3 569w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777384365; x=1777989165; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=TvN+aQrEQG1INNwJXWKA667EIqAzFBBNzUYu75gRDbM=; b=JT3kNNh0dptPqHJLm7JilMCzlE6oX6lF4/5mh0sEEXeIhCZnSfXGBR6jm6nqth0wsE UC89HEC2k0PsUcH+rI0dwYcbqJginqa0DuHhud+ZWgN3gYMWhz0qKGQZa2Ln79UbbWve DdFc7e47e7Bmh10twjPtS69INgKqaD/WVmX+EiOUS6z7ueHcSAD/xJu15nfSw3KFdBN+ kQ8JR2inMcNBs5uXz3mOLdbyEutRWvcGUKxdM4mxpGRfH+eUJxabV3jz2dKh0tZTpBsz n81rBze7C1H8frj0RKSIXHZtyOsEeu8bPusYxZ/ciW/OEOFnW+k7X+pSITQZNCTzcU6+ YTtQ== X-Forwarded-Encrypted: i=1; AFNElJ8PzEHsrRYh/B1dfTdGsZr/LWhpXzDxCKwceBZpw9JAGUVpySUHN+UPaPCXk1B+scJRIqj/Mf/JPaZkytQ=@vger.kernel.org X-Gm-Message-State: AOJu0Yzz9yon+beMQdtXTiUgtuC4tG29XNhPyQXmjcPBUqb7XhsQE6eG Eoa8M3csuvMgu7PSgcRFOxv5hz9jIJCRyP/F3LXYdJxnOt0F2CmsFK8r X-Gm-Gg: AeBDiesB2WZRj2wfBOprHt7yNpK2HYscdSEb/qGbkY5vNchOLK/y3++FojCZBT+sTCj mWKQRFoc0nGdksOyGkC2YVaxcl/Eb34pIYUzU1Hvy3KyzXaIGblW9/hr7ldxBcX9EUNZe8RQWGD ZRC5TNGAa4DZixHm5Ryc+qA/+zRHpMkuI4Uqj5hLtzDAOo6R1annI/Yg1pD5b7ccK5UJyqtd3H9 kCMNxBWljWxF4+32Cdzwr3nQ1lvyzoV3igW5rUE3BbUK/TsN/o9BR32i5LX7ylOcfIN1vkeAKs+ YdJXi8JxqZ1NvZjLksMJHhmjks/bMvhtyzUS8nV10A0bO5bMVl0dLKv8whrmTvnt2NzOXkNdzqo RY8shUFbiG+RD2zQKKmVT3Se7OgnmQSGLxIbAB0+2gTDGDiHBjbWSupuBlOIV3zgU+nOWXgwpwT 978Lw3SnEtuU2gKq5ZISVs5pQMCExc8QqhQT3bVw3D X-Received: by 2002:a05:600c:1396:b0:488:8577:d9cc with SMTP id 5b1f17b1804b1-48a77b0ed61mr53199745e9.20.1777384365202; Tue, 28 Apr 2026 06:52:45 -0700 (PDT) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-4463f4c0b0dsm7062593f8f.23.2026.04.28.06.52.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 28 Apr 2026 06:52:44 -0700 (PDT) Date: Tue, 28 Apr 2026 16:52:41 +0300 From: Dan Carpenter To: Ricardo Ribalda Cc: Mauro Carvalho Chehab , Laurent Pinchart , Sakari Ailus , Hans Verkuil , Nas Chung , Jackson Lee , Bingbu Cao , Tianshu Qiu , Greg Kroah-Hartman , Keke Li , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, linux-staging@lists.linux.dev Subject: Re: [PATCH 0/6] media: Fix new smatch warnings Message-ID: References: <20260428-smatch-7-1-v1-0-46890dffb611@chromium.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260428-smatch-7-1-v1-0-46890dffb611@chromium.org> On Tue, Apr 28, 2026 at 12:41:06PM +0000, Ricardo Ribalda wrote: > Current version of smatch triggers some warnings for the media tree. > Most of them are inoffensive, but we would like to have zero smatch > warnings. > > drivers/media/platform/amlogic/c3/isp/c3-isp-params.c:111 c3_isp_params_awb_wt() error: buffer overflow 'cfg->zone_weight' 768 <= u32max > drivers/media/platform/amlogic/c3/isp/c3-isp-params.c:111 c3_isp_params_awb_wt() error: buffer overflow 'cfg->zone_weight' 768 <= u32max > drivers/media/platform/amlogic/c3/isp/c3-isp-params.c:227 c3_isp_params_ae_wt() error: buffer overflow 'cfg->zone_weight' 255 <= u32max > drivers/media/platform/amlogic/c3/isp/c3-isp-params.c:227 c3_isp_params_ae_wt() error: buffer overflow 'cfg->zone_weight' 255 <= u32max > drivers/media/v4l2-core/v4l2-dev.c:1036 __video_register_device() error: buffer overflow 'video_devices' 256 <= 288 > drivers/media/v4l2-core/v4l2-dev.c:1043 __video_register_device() error: buffer overflow 'video_devices' 256 <= 288 > drivers/media/v4l2-core/v4l2-dev.c:1101 __video_register_device() error: buffer overflow 'video_devices' 256 <= 288 > drivers/media/platform/chips-media/wave5/wave5-vpuapi.c:588 wave5_vpu_dec_get_output_info() error: buffer overflow 'inst->frame_buf' 64 <= 127 > drivers/staging/media/ipu3/ipu3-css-params.c:1792 imgu_css_cfg_acc_stripe() warn: 'acc->stripe.bds_out_stripes[0]->width - 2 * f' 4294967168 can't fit into 65535 'acc->stripe.bds_out_stripes[1]->offset' > drivers/media/i2c/adv7604.c:3672 adv76xx_probe() error: buffer overflow 'state->pads' 7 <= 4294967294 > drivers/media/i2c/adv7604.c:3673 adv76xx_probe() error: buffer overflow 'state->pads' 7 <= u32max > drivers/media/i2c/mt9p031.c:799 mt9p031_s_ctrl() warn: assigning (-1952) to unsigned variable 'data' > > Signed-off-by: Ricardo Ribalda I'm re-writing a bunch of core stuff right now... Feel free to complain about false positives. I'm going to re-write the buffer overflow warning in the next couple weeks. regards, dan carpenter