st/omx: don't return early in vid_enc_EncodeFrame()
authorEmil Velikov <emil.velikov@collabora.com>
Fri, 29 Apr 2016 12:51:45 +0000 (13:51 +0100)
committerEmil Velikov <emil.l.velikov@gmail.com>
Fri, 29 Apr 2016 14:36:18 +0000 (15:36 +0100)
commit4b1ea6910ee54afb30fd005eb1f8cf6f88338eda
tree15e51c860b82b39a361cefe9de7f09b633acdfe5
parentc750029b37c64521267910af0ec2c89c58c82987
st/omx: don't return early in vid_enc_EncodeFrame()

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")
Tested-by: Andy Furniss <adf.lists@gmail.com>
Acked-by: Robert Foss <robert.foss@collabora.com>
Signed-off-by: Emil Velikov <emil.velikov@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