[PATCH v2 0/7] gstreamer: Miscellaneous cleanups + two fixes

[Laurent Pinchart]

On Wed, Jun 04, 2025 at 04:07:34PM +0300, Laurent Pinchart wrote:
> Hello everybody,
> 
> Here's a set of miscellaneous cleanups for libcamerasrc that follow the
> recently added GstVideoMeta support. They mostly improve readability,
> but patches 5/7 and 6/7 also fix two leaks and a use-after-free
> respectively.
> 
> Compared to v1, I've addressed review comments, reordered the patches,
> added more leak and use-after-free fixes, added patch 7/7, and performed
> other small changes as detailed in the per-patch changelogs.
> 
> The code has still been compile-tested only as I haven't managed to
> exercize the video pool creation code path yet. Hou, would you be able
> to test this series ?

Pending a review of patch 7/7, this series is ready to be merged. Hou,
could you please let me know if you plan to test it ? If so I'll wait
for your test report before merging.

> Laurent Pinchart (7):
>   gstreamer: Document improvements when updating minimum GStreamer
>     version
>   gstreamer: Factor out video pool creation
>   gstreamer: Reduce indentation in gst_libcamera_create_video_pool()
>   gstreamer: Rename variable in gst_libcamera_create_video_pool()
>   gstreamer: Fix leak of GstQuery and GstBufferPool in error path
>   gstreamer: Drop incorrect unref on caps
>   gstreamer: Replace NULL with nullptr
> 
>  src/gstreamer/gstlibcamera-controls.cpp.in |   2 +-
>  src/gstreamer/gstlibcamerapad.cpp          |   2 +-
>  src/gstreamer/gstlibcamerasrc.cpp          | 146 ++++++++++++++-------
>  3 files changed, 98 insertions(+), 52 deletions(-)
> 
> 
> base-commit: 290d3f82e3e3e313932a201308813c75131d16f1

-- 
Regards,

Laurent Pinchart