media: uvcvideo: Do not mark valid metadata as invalid
authorRicardo Ribalda <ribalda@chromium.org>
Mon, 7 Jul 2025 18:34:01 +0000 (18:34 +0000)
committerHans Verkuil <hverkuil@xs4all.nl>
Fri, 11 Jul 2025 17:27:28 +0000 (19:27 +0200)
commitbda2859bff0b9596a19648f3740c697ce4c71496
tree60e13f1a60f4898fc87b8fc3b566cf18f03d0193
parent2acded8aa2c89b858495488f6a14cd1d04fc5784
media: uvcvideo: Do not mark valid metadata as invalid

Currently, the driver performs a length check of the metadata buffer
before the actual metadata size is known and before the metadata is
decided to be copied. This results in valid metadata buffers being
incorrectly marked as invalid.

Move the length check to occur after the metadata size is determined and
is decided to be copied.

Cc: stable@vger.kernel.org
Fixes: 088ead255245 ("media: uvcvideo: Add a metadata device node")
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Hans de Goede <hansg@kernel.org>
Signed-off-by: Ricardo Ribalda <ribalda@chromium.org>
Link: https://lore.kernel.org/r/20250707-uvc-meta-v8-1-ed17f8b1218b@chromium.org
Signed-off-by: Hans de Goede <hansg@kernel.org>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
drivers/media/usb/uvc/uvc_video.c