{"id":2233304,"url":"http://patchwork.ozlabs.org/api/1.2/patches/2233304/?format=json","web_url":"http://patchwork.ozlabs.org/project/linux-tegra/patch/20260506075745.28574-2-dennylin0707@gmail.com/","project":{"id":21,"url":"http://patchwork.ozlabs.org/api/1.2/projects/21/?format=json","name":"Linux Tegra Development","link_name":"linux-tegra","list_id":"linux-tegra.vger.kernel.org","list_email":"linux-tegra@vger.kernel.org","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20260506075745.28574-2-dennylin0707@gmail.com>","list_archive_url":null,"date":"2026-05-06T07:57:45","name":"[v2,1/1] media: tegra-video: vi: fix invalid u32 return value in format lookup","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"f165c290f5bd1194538b9687e7380dde12250f96","submitter":{"id":93122,"url":"http://patchwork.ozlabs.org/api/1.2/people/93122/?format=json","name":"Hungyu Lin","email":"dennylin0707@gmail.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/linux-tegra/patch/20260506075745.28574-2-dennylin0707@gmail.com/mbox/","series":[{"id":502930,"url":"http://patchwork.ozlabs.org/api/1.2/series/502930/?format=json","web_url":"http://patchwork.ozlabs.org/project/linux-tegra/list/?series=502930","date":"2026-05-06T07:57:44","name":"media: tegra-video: vi: fix invalid u32 return value in format lookup","version":2,"mbox":"http://patchwork.ozlabs.org/series/502930/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/2233304/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2233304/checks/","tags":{},"related":[],"headers":{"Return-Path":"\n <linux-tegra+bounces-14231-incoming=patchwork.ozlabs.org@vger.kernel.org>","X-Original-To":["incoming@patchwork.ozlabs.org","linux-tegra@vger.kernel.org"],"Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256\n header.s=20251104 header.b=q9hTC8TI;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=2600:3c04:e001:36c::12fc:5321; helo=tor.lore.kernel.org;\n envelope-from=linux-tegra+bounces-14231-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","smtp.subspace.kernel.org;\n\tdkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com\n header.b=\"q9hTC8TI\"","smtp.subspace.kernel.org;\n arc=none smtp.client-ip=74.125.82.194","smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=gmail.com","smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=gmail.com"],"Received":["from tor.lore.kernel.org (tor.lore.kernel.org\n [IPv6:2600:3c04:e001:36c::12fc:5321])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4g9SRk1nkwz1yKd\n\tfor <incoming@patchwork.ozlabs.org>; Wed, 06 May 2026 17:58:10 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby tor.lore.kernel.org (Postfix) with ESMTP id 51F1B300C9AF\n\tfor <incoming@patchwork.ozlabs.org>; Wed,  6 May 2026 07:58:07 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 3366C3E1CF8;\n\tWed,  6 May 2026 07:58:06 +0000 (UTC)","from mail-dy1-f194.google.com (mail-dy1-f194.google.com\n [74.125.82.194])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id B21123E0C54\n\tfor <linux-tegra@vger.kernel.org>; Wed,  6 May 2026 07:58:04 +0000 (UTC)","by mail-dy1-f194.google.com with SMTP id\n 5a478bee46e88-2ee990e8597so6472577eec.1\n        for <linux-tegra@vger.kernel.org>;\n Wed, 06 May 2026 00:58:04 -0700 (PDT)","from localhost.localdomain ([76.32.119.210])\n        by smtp.gmail.com with ESMTPSA id\n 5a478bee46e88-2f56d4cf2fdsm2416561eec.8.2026.05.06.00.58.02\n        (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n        Wed, 06 May 2026 00:58:03 -0700 (PDT)"],"ARC-Seal":"i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1778054286; cv=none;\n b=gt3MFr3hWABjx8KF+giT91+veFmbfxKqnm+6DB65X2XvXn0jq0PMYk3GaCa1ZTMbWPcZ7sAkmWRTCem+gOq84ZCrR/+SKTLX7ksXQ9VqIMthebj5OCeCkeweIvgKWSspi8RSP6nmwacZSq1SCqikfvRIfjMtbDvMjQDHabL8QQ0=","ARC-Message-Signature":"i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1778054286; c=relaxed/simple;\n\tbh=NxDn3Yx28HJtPbWuriUuKQ1bU4FkzVJ+n8JDBIb0o2I=;\n\th=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References:\n\t MIME-Version;\n b=VUabYYm8+LNODiqO1zCEY4qB5ioVMV4lTki4PpnnkIoXIeLub1xcvKPVyDrPAHolAJ55JG8WFXpA4H2xLPvNhFqsTH0/4ljL/h0PE0GcrTZDBXBPaXvJpEtQA72TGDSihY62i9pwkaarDQ7hdIuD9WoLgRcJE9f/RKE3NZPT/n8=","ARC-Authentication-Results":"i=1; smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=gmail.com;\n spf=pass smtp.mailfrom=gmail.com;\n dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com\n header.b=q9hTC8TI; arc=none smtp.client-ip=74.125.82.194","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n        d=gmail.com; s=20251104; t=1778054284; x=1778659084;\n darn=vger.kernel.org;\n        h=content-transfer-encoding:mime-version:references:in-reply-to\n         :message-id:date:subject:cc:to:from:from:to:cc:subject:date\n         :message-id:reply-to;\n        bh=0yP+mwFYhjefiJc9HNwtug4g60Bu6O6itewJ8verryo=;\n        b=q9hTC8TIg2Uvbz2B18ecXgVKwj25Tc1ez3DbAA9ZeizaGQbAhnSZPHqZ+i/8y0ESDI\n         buKGBdTKEh2sYMjk+S5W+Emb9Ym0mqd6jC0Y2+9QTdIcPM37JYx3W8iKSQrAMtNLGc92\n         3lFgH12Lx7OPV2B2ag5Wz6Tk8ZLWYMK/PCOpS3CeP+D999I0eWUnKXF4KO+QotjxcfMC\n         qcjkUlZdRe0pTroYJcgY7RudcaXFfy0EH6dzggPoDOEURHKlrGvhZZPvTR0vFd/wK6dm\n         OBtlu2bnQ5f7y4BIpQgDgiace6IqL4YYQM6LOAgTijVtd/qI53S8qdqpAQMebfdkIVJS\n         i38w==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n        d=1e100.net; s=20251104; t=1778054284; x=1778659084;\n        h=content-transfer-encoding:mime-version:references:in-reply-to\n         :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from\n         :to:cc:subject:date:message-id:reply-to;\n        bh=0yP+mwFYhjefiJc9HNwtug4g60Bu6O6itewJ8verryo=;\n        b=iBCb0Wcu7TkM6Upit1ZMV8ntXJtrCzb6v9NpuusTIRDHJ8XoiDaI3ZA2BmUZLCxfym\n         U15KtntOp0OGpvDAS4Ca/SMeY+V9qBAF1gZXMAgwmfysi9Ow51wSbs6Pr96gpA/zdJqv\n         ge3YMFPiAsEKRNWUC/lRUH/F91J7O1LqLQRue+ledwR+KERdhC09Hft3Li5u0PHBuWeu\n         mkHGC1k4Fa2vcCfV4T97g7jcl6UlfoKPsskmqTvfm1YPwpL990SxXTB04Igpw9ZNVisd\n         f8X1akqqDRUVWcMbI5JIVb7SOgaAppk/NhB/JxoQewOt40oPtr+pfcgIfrbLyITLi5/u\n         gKqQ==","X-Forwarded-Encrypted":"i=1;\n AFNElJ+NvAERYukKSIGd98i//mXBs1UCkgWPdWotfeGH0VhlGPyyY9VY10kPrS73RvGBwlZOj03sUygQw2aR/g==@vger.kernel.org","X-Gm-Message-State":"AOJu0Yz55PFKevdGe6Hw3X5ksUH6n9ocT21EgUNAJtXpvc0mmn1qaR6o\n\tF8b2fkKoU0dXSOYuQODOhW5RJYATF2K6asy4YJH98d8mu1RYUYQegl1I","X-Gm-Gg":"AeBDiety/rwVHRuMpvyE5Jz38BE/EExLV3XH/9TWlS1JnzE6pcwFhdEcYD9NV1cGOZa\n\tECqWVUWbjwh07iLp3FD3+kZQb181Bn/og99UMDaQN2Kf4Orqv1U44i2ZSj98Hd8UT5ZX3iI6Cni\n\txhR3LjZ8JrQK7PUfdSdQeUbtoYi6LJgTy4tw8RFNkxhdVJBCvADLKnFhrxAhFbbUhH92EMhqkd9\n\t/YtirAFF1KeQ7Ft70qdg4nc6xSv97u8Mnc82hdSJHz9Rq970Yus6oMKO6qCqxN4dQbvfNT8NQEv\n\t3Sck1P//weUqrnxvdeXn+ACTvNmBZ+7Zp6Rn5Io3t5I1DMrNCpwXN9JB5yDpePGZMrGdCgh8iO+\n\tGxlMxfUN+4a6IqUN9iyoJceEzGfyCU5JOy4c4NdnOklDtGkTf1NZ/Lp+hzl57LcBOY7fW6xa4ho\n\tjya3rutnUoS8OrvLrc3XffksVExCzw6X8FdYCnzwtMzkutbl8g8rQSbVQrxG4mbcmH7w==","X-Received":"by 2002:a05:7300:4794:b0:2f2:5726:db4e with SMTP id\n 5a478bee46e88-2f54ac74abcmr1120763eec.18.1778054283787;\n        Wed, 06 May 2026 00:58:03 -0700 (PDT)","From":"Hungyu Lin <dennylin0707@gmail.com>","To":"thierry.reding@kernel.org,\n\tjonathanh@nvidia.com,\n\tskomatineni@nvidia.com,\n\tluca.ceresoli@bootlin.com,\n\tmchehab@kernel.org,\n\tgregkh@linuxfoundation.org","Cc":"hverkuil+cisco@kernel.org,\n\tdan.carpenter@linaro.org,\n\tlinux-media@vger.kernel.org,\n\tlinux-tegra@vger.kernel.org,\n\tlinux-staging@lists.linux.dev,\n\tlinux-kernel@vger.kernel.org,\n\tHungyu Lin <dennylin0707@gmail.com>","Subject":"[PATCH v2 1/1] media: tegra-video: vi: fix invalid u32 return value\n in format lookup","Date":"Wed,  6 May 2026 07:57:45 +0000","Message-Id":"<20260506075745.28574-2-dennylin0707@gmail.com>","X-Mailer":"git-send-email 2.34.1","In-Reply-To":"<20260506075745.28574-1-dennylin0707@gmail.com>","References":"<20260506075745.28574-1-dennylin0707@gmail.com>","Precedence":"bulk","X-Mailing-List":"linux-tegra@vger.kernel.org","List-Id":"<linux-tegra.vger.kernel.org>","List-Subscribe":"<mailto:linux-tegra+subscribe@vger.kernel.org>","List-Unsubscribe":"<mailto:linux-tegra+unsubscribe@vger.kernel.org>","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit"},"content":"tegra_get_format_fourcc_by_idx() returns a u32 but uses -EINVAL to\nsignal an out-of-bounds index. This results in a large unsigned\nvalue being returned, which may be interpreted as a valid fourcc.\n\nReturning 0 is not a valid fourcc either. This condition should\nnever happen, so use WARN_ON_ONCE() to catch unexpected out-of-bounds\naccess and return a valid fallback format instead.\n\nSuggested-by: Hans Verkuil <hverkuil+cisco@kernel.org>\nSigned-off-by: Hungyu Lin <dennylin0707@gmail.com>\n---\n drivers/staging/media/tegra-video/vi.c | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)","diff":"diff --git a/drivers/staging/media/tegra-video/vi.c b/drivers/staging/media/tegra-video/vi.c\nindex f14cdc7b5211..456134a9e8cf 100644\n--- a/drivers/staging/media/tegra-video/vi.c\n+++ b/drivers/staging/media/tegra-video/vi.c\n@@ -80,8 +80,8 @@ static int tegra_get_format_idx_by_code(struct tegra_vi *vi,\n static u32 tegra_get_format_fourcc_by_idx(struct tegra_vi *vi,\n \t\t\t\t\t  unsigned int index)\n {\n-\tif (index >= vi->soc->nformats)\n-\t\treturn -EINVAL;\n+\tif (WARN_ON_ONCE(index >= vi->soc->nformats))\n+\t\treturn vi->soc->video_formats[0].fourcc;\n \n \treturn vi->soc->video_formats[index].fourcc;\n }\n","prefixes":["v2","1/1"]}