Received-SPF: pass (google.com: domain of linux-kernel+bounces-191624-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: Nas Chung <nas.chung@chipsnmedia.com>
To: mchehab@kernel.org,
	linux-media@vger.kernel.org,
	sebastian.fricke@collabora.com,
	m.tretter@pengutronix.de
Cc: linux-kernel@vger.kernel.org,
	Nas Chung <nas.chung@chipsnmedia.com>
Subject: [PATCH v2] media: uapi: v4l: Change V4L2_TYPE_IS_CAPTURE condition
Date: Tue, 28 May 2024 11:04:25 +0900
Message-Id: <20240528020425.4994-1-nas.chung@chipsnmedia.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?23RZm70uVP8AcxFqhEiEbb2e52ksQ0eCxRfnNYVkq5JdWEvzIBKbU06c2BBN?=
 =?us-ascii?Q?lYchLc1OROCK2lFuLhZCUX+nz3Kg2zQmALB2CM2iiB7fJE6sPwdY2n2ohiRp?=
 =?us-ascii?Q?4U8aoIdlMN7OskcY7iHPY2Nb+3IYeN24QsszFYrWL96xvZ8pF26TY61q2JMO?=
 =?us-ascii?Q?nXkWnw7ui2LBh9PhpsEIOJ5Yih2PUXjylnzItJCeMiW5ZdIi32BcazAuQeiw?=
 =?us-ascii?Q?2KAl6lHyHwf0HLoO8FERkfmk5xLDJ7nIVC1KtLvvOOFn3WCPNMhq3yZgjcz1?=
 =?us-ascii?Q?dQB1+knJINvBODiU4rwR1fNYXJdckZiJO3tsW75QE+Sh/v9fhaQ99HvCzdOx?=
 =?us-ascii?Q?58oBTeO+afzqzdG5XsCqhOt9NJOpvZ975YrnlNRWZQZBq4S4Qa9bD8/iI/DM?=
 =?us-ascii?Q?1OdoKLvfyzDgVCsO/ksRfuNPXARUENGCYn3bwmH5rCs+jDkYJvePEbu8z7K6?=
 =?us-ascii?Q?/quUxQq5SHfeN+r2XTqI0p6RzA0r8bMmA2PD7hzoP7zqooD1qt3GocPIx/10?=
 =?us-ascii?Q?yTyMDKs9EsOBRRB4TzFGxw+XQNP9iNIY/PA4SSIRwQ27Uv2kSb7AWmdFsJAV?=
 =?us-ascii?Q?IzSx4m0+j2tMRGGTzjEws8Gw7/HOIrJSxcsZjqDm34xF5ZckzVOcX1yJVqIZ?=
 =?us-ascii?Q?sxqIXIi1wZvxvqg2f5t83DsE7G7K52GPKWPVndlJDbPTalq/LWEP0tpjKBw8?=
 =?us-ascii?Q?orXprtXHFE7EBf6CvP+80bLvCE2oVle63eNlpF2lALTGmcLqL7kVlaWaYc4h?=
 =?us-ascii?Q?4idrwrwS5fhG0Ues89rtSUqXGYAkKu/LARAghdKx06n+VBuUuE8w9v4HqLTL?=
 =?us-ascii?Q?fZXJBuCUd2KtWSHbbDReR8McRaA9H4i4wYKLemjpw9fjvanvx16oe/46fKTz?=
 =?us-ascii?Q?I9vCpLU3txfLBiBNAr6EaitgZlBn1L7pvlZbi0Y5ZT+KLb3Q2PrGgqjUrhCa?=
 =?us-ascii?Q?1xHuKiWkZvCg9QeaeFcz5wk0RbDukk7Hremkr1+dij29Fp9BkSBibrYDK+Hn?=
 =?us-ascii?Q?SYaBdLrVwxiAJ9dNp9h8VKJXLzOIVZD/vNNQxWkYB8mnr0pZf1yNsX3n8tAa?=
 =?us-ascii?Q?GeWBiizNeioHVehqmDd3j1JMOLPiHS3zxeVjP6lY0M7crQnnKLPP0Yqrg0Q6?=
 =?us-ascii?Q?z8lM4qPic5QmD4QzhvOtYRM86PG2wyMzH6TuLVONOEhfdcLOhTVaS+7P9uAt?=
 =?us-ascii?Q?sJyXLRR5bJjQUvFB16CYOyL1aRrdE4Q4ZlVGoJLSoLCP/yzIuhXNPnHAwVx9?=
 =?us-ascii?Q?LM70/aA3kU39/vwbHvzG6FeCtQREPIWosRwNIK6jRVIr5xfzTjxq8qXvSqd1?=
 =?us-ascii?Q?ANIcvPoaE/DyeNpnuwUJyICEJzcRrDoQELgidk7SSbkrZ/VdPBIpJgI+mKF6?=
 =?us-ascii?Q?+sVefNeSXWmFT1pp7loK1lOLkCJPJp1GuQeghDom60iAJi7Q9FIg+Sa76HvA?=
 =?us-ascii?Q?PKcYBfSI/JjZXjpbN/RI7x3QkattWMDClYNHr9wOel9pe7dg9GRTjUY5JkUw?=
 =?us-ascii?Q?tHOXpgCLNw5TMTHNXd/g9nQ8dN+vG1yPBQMYDhc03Jo1SHMRozuJ5lqeqVxy?=
 =?us-ascii?Q?BQpu+qBbcVme1J+eU4g7mpbfWJFbRRaCgABObCn2+zsPMjujU5HWU5CN/DmW?=
 =?us-ascii?Q?fg=3D=3D?=
X-OriginatorOrg: chipsnmedia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1613553d-d29f-44d1-96f4-08dc7eba84ef
X-MS-Exchange-CrossTenant-AuthSource: SL2P216MB1246.KORP216.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 May 2024 02:04:34.0282
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4d70c8e9-142b-4389-b7f2-fa8a3c68c467
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: CzPOYR7bg8f1s8maAr+Z7A6ST3gW/qfnA93F/vjM2VY+mYz0WXCUwwXIY5DK4j2la46/+hjvHysfbQCrC5zT4rdCKLaJCvF/OfVlEBnAX+A=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PS2P216MB1387

Explicitly compare a buffer type only with valid buffer types,
to avoid matching the buffer type outside of valid buffer
type set.

Signed-off-by: Nas Chung <nas.chung@chipsnmedia.com>
---
 include/uapi/linux/videodev2.h | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h
index fe6b67e83751..fa2b7086e480 100644
--- a/include/uapi/linux/videodev2.h
+++ b/include/uapi/linux/videodev2.h
@@ -157,6 +157,10 @@ enum v4l2_buf_type {
 	V4L2_BUF_TYPE_PRIVATE              = 0x80,
 };
 
+#define V4L2_TYPE_IS_VALID(type)		\
+	((type) >= V4L2_BUF_TYPE_VIDEO_CAPTURE	\
+	 && (type) <= V4L2_BUF_TYPE_META_OUTPUT)
+
 #define V4L2_TYPE_IS_MULTIPLANAR(type)			\
 	((type) == V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE	\
 	 || (type) == V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE)
@@ -171,7 +175,8 @@ enum v4l2_buf_type {
 	 || (type) == V4L2_BUF_TYPE_SDR_OUTPUT			\
 	 || (type) == V4L2_BUF_TYPE_META_OUTPUT)
 
-#define V4L2_TYPE_IS_CAPTURE(type) (!V4L2_TYPE_IS_OUTPUT(type))
+#define V4L2_TYPE_IS_CAPTURE(type)	\
+	(V4L2_TYPE_IS_VALID(type) && !V4L2_TYPE_IS_OUTPUT(type))
 
 enum v4l2_tuner_type {
 	V4L2_TUNER_RADIO	     = 1,
-- 
2.25.1