diff options
| author | Barnabás Pőcze <pobrn@protonmail.com> | 2024-05-14 18:02:07 +0000 |
|---|---|---|
| committer | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2024-05-14 23:51:57 +0300 |
| commit | e43d2a35fa671c18dd8bae39ab21df9671b5f0a1 (patch) | |
| tree | a4b0d33d4d2ccc137bad069b383e860aa5305fb5 /src/gstreamer/gstlibcameraallocator.cpp | |
| parent | f113464b3481d76dee158a00c790b44671b382b7 (diff) | |
gstreamer: Use copied camera name
The camera name is copied in gst_libcamera_src_open() as we can't hold
the lock protecting the name while calling GST_ELEMENT_ERROR(). The
GST_ELEMENT_ERROR() macro sends a message on the bus, taking more locks
and possibly causing issues.
However, the current code makes the copy, but does not actually use it.
So fix that.
Fixes: 58feb69f852289 ("gst: libcamerasrc: Implement selection and acquisition")
Signed-off-by: Barnabás Pőcze <pobrn@protonmail.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Diffstat (limited to 'src/gstreamer/gstlibcameraallocator.cpp')
0 files changed, 0 insertions, 0 deletions