diff options
author | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2021-06-17 02:15:40 +0300 |
---|---|---|
committer | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2021-06-17 12:37:23 +0300 |
commit | 73b823b22009b87fc1d33a8000a870fe9223e7bb (patch) | |
tree | ce3b5939f62e2b2619e6d5f84a8427ef590e1f78 /test/v4l2_videodevice/double_open.cpp | |
parent | b4c3db4c015082a2866e3e546b74d3b3de253d7f (diff) |
libcamera: pipeline: simple: Fix crash when storing timestamp in metadata
Commit 922833f774f6 ("libcamera: simple: Report sensor timestamp")
unconditionally tries to access the request through the capture buffer
to store the capture timestamp in the metadata. This causes a null
pointer dereference when using a converter, as the capture buffers are
free-wheeling in that case, and not associated with a request.
Fix this by getting the request from the user-facing buffer, which can
be the capture buffer when no converter is used.
Fixes: 922833f774f6 ("libcamera: simple: Report sensor timestamp")
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Niklas Söderlund <niklas.soderlund@ragnatech.se>
Reviewed-by: Jacopo Mondi <jacopo@jmondi.org>
Diffstat (limited to 'test/v4l2_videodevice/double_open.cpp')
0 files changed, 0 insertions, 0 deletions