[Mesa-dev] [PATCH] st/omx: don't return early in vid_enc_EncodeFrame()

[Emil Velikov]

From: Emil Velikov <emil.velikov at collabora.com>

Earlier commit plugged a memory leak, although it missed a pair of
brackets. Thus we unconditionally returned even in the case of no error.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95203
Fixes: b87856d25d1 ("st/omx: Fix resource leak on OMX_ErrorNone")
Cc: Andy Furniss <adf.lists at gmail.com>
Cc: Robert Foss <robert.foss at collabora.com>
Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
---
What an embarassing bug - missing brackets. Andy can you confirm that it
resolves the issue ?
---
 src/gallium/state_trackers/omx/vid_enc.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/gallium/state_trackers/omx/vid_enc.c b/src/gallium/state_trackers/omx/vid_enc.c
index de62166..5565241 100644
--- a/src/gallium/state_trackers/omx/vid_enc.c
+++ b/src/gallium/state_trackers/omx/vid_enc.c
@@ -1151,9 +1151,10 @@ static OMX_ERRORTYPE vid_enc_EncodeFrame(omx_base_PortType *port, OMX_BUFFERHEAD
    } else {
       /* ------- load input image into video buffer ---- */
       err = enc_LoadImage(port, buf, task->buf);
-      if (err != OMX_ErrorNone)
+      if (err != OMX_ErrorNone) {
          FREE(task);
          return err;
+      }
    }
 
    /* -------------- determine picture type --------- */
-- 
2.6.2