diff options
| author | Guillaume Desmottes <guillaume.desmottes@collabora.co.uk> | 2017-07-27 12:12:01 +0200 |
|---|---|---|
| committer | Nicolas Dufresne <nicolas.dufresne@collabora.com> | 2017-08-01 11:26:11 -0400 |
| commit | 7a0949b7501ee226e6d59c75136b97cae8591380 (patch) | |
| tree | 7cba1926a5eba4228f11bce06e3100b032f2adf1 /omx/gstomx.c | |
| parent | 86d9a2c81c60a3a246c7052e70e9d50b8a12b156 (diff) | |
| download | gst-omx-7a0949b7501ee226e6d59c75136b97cae8591380.tar.gz | |
omx: also reset nFilledLen before calling OMX_FillThisBuffer()
The spec states that the buffer passed to OMX_FillThisBuffer() needs to be
empty. Some implementation may check it actually is by checking its
nFilledLen field, so best to reset it as well.
https://bugzilla.gnome.org/show_bug.cgi?id=785623
Diffstat (limited to 'omx/gstomx.c')
| -rw-r--r-- | omx/gstomx.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/omx/gstomx.c b/omx/gstomx.c index cf841e6..191b719 100644 --- a/omx/gstomx.c +++ b/omx/gstomx.c @@ -2078,9 +2078,11 @@ gst_omx_port_populate_unlocked (GstOMXPort * port) /* Reset all flags, some implementations don't * reset them themselves and the flags are not - * valid anymore after the buffer was consumed + * valid anymore after the buffer was consumed. + * Also reset nFilledLen as FillThisBuffer() expects an empty buffer. */ buf->omx_buf->nFlags = 0; + buf->omx_buf->nFilledLen = 0; err = OMX_FillThisBuffer (comp->handle, buf->omx_buf); |