Skip to content

Commit 8a403fc

Browse files
srikarrikhusika
authored andcommitted
mm-video:don't remove 0 timestamp buffer for config buffer
-For codec config buffer, 0 time stamp buffer is been removed from timestamp queue which causes cts case testCodecBasicH264 to fail. -Fix:Do not remove 0 time stamp buffer from time stamp queue for codec config buffer case Change-Id: I57a0f1e778f65997170c748a17201fc8a28b3cff
1 parent b40a565 commit 8a403fc

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

mm-video/vidc/vdec/src/omx_vdec.cpp

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -916,11 +916,15 @@ void omx_vdec::process_event_cb(void *ctxt, unsigned char id)
916916
{
917917
if (p2 == VDEC_S_INPUT_BITSTREAM_ERR && p1)
918918
{
919-
pThis->m_inp_err_count++;
920-
pThis->time_stamp_dts.remove_time_stamp(
921-
((OMX_BUFFERHEADERTYPE *)p1)->nTimeStamp,
922-
(pThis->drv_ctx.interlace != VDEC_InterlaceFrameProgressive)
923-
?true:false);
919+
if (!(((OMX_BUFFERHEADERTYPE *)p1)->nFlags & OMX_BUFFERFLAG_CODECCONFIG))
920+
{
921+
DEBUG_PRINT_HIGH("remove the timestamp from queue for VDEC_S_INPUT_BITSTREAM_ERR");
922+
pThis->m_inp_err_count++;
923+
pThis->time_stamp_dts.remove_time_stamp(
924+
((OMX_BUFFERHEADERTYPE *)p1)->nTimeStamp,
925+
(pThis->drv_ctx.interlace != VDEC_InterlaceFrameProgressive)
926+
?true:false);
927+
}
924928
}
925929
else
926930
{

0 commit comments

Comments
 (0)