Gst_dsexample_transform_ip() from the example plugin is not properly handling errors

DeepStream Version: 5.0

In the aforementioned function there are all kinds of problems with how the function handles errors. If gst_buffer_map() fails it does a ‘goto error’ where it will try to call gst_buffer_unmap(). I don’t know what the implications are of calling that with memory that was not successfully mapped. And, then there are many places in the function where on error it will simply ‘return GST_FLOW_ERROR’ w/o having first calling gst_buffer_unmap(). It should just call ‘goto error’ for proper cleanup.

We will review our codes to check how to improve it. Thank you!