Commit 87b1084c authored by Derk-Jan Hartman's avatar Derk-Jan Hartman

TS demux: Clarify a for() loop.

This will likely fix CID 247, which seems like a false.
I don't see how PIDClean is called twice here, I think it's a limitation in the Coverity scan logic.
parent 34aef4ad
...@@ -4021,7 +4021,7 @@ static void PATCallBack( demux_t *p_demux, dvbpsi_pat_t *p_pat ) ...@@ -4021,7 +4021,7 @@ static void PATCallBack( demux_t *p_demux, dvbpsi_pat_t *p_pat )
if( !pid->b_valid || pid->psi ) continue; if( !pid->b_valid || pid->psi ) continue;
for( j = 0; j < i_pmt_rm; j++ ) for( j = 0; j < i_pmt_rm && pid->b_valid; j++ )
{ {
int i_prg; int i_prg;
for( i_prg = 0; i_prg < pid->p_owner->i_prg; i_prg++ ) for( i_prg = 0; i_prg < pid->p_owner->i_prg; i_prg++ )
...@@ -4041,8 +4041,6 @@ static void PATCallBack( demux_t *p_demux, dvbpsi_pat_t *p_pat ) ...@@ -4041,8 +4041,6 @@ static void PATCallBack( demux_t *p_demux, dvbpsi_pat_t *p_pat )
PIDClean( p_demux->out, pid ); PIDClean( p_demux->out, pid );
break; break;
} }
if( !pid->b_valid ) break;
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment