Commit 01423a06 authored by François Cartegnie's avatar François Cartegnie 🤞

demux: ts: fix use after free

regression from 48edcfab,
missing return in refactored code
parent e2805ca8
...@@ -1587,7 +1587,7 @@ static bool PushPESBlock( demux_t *p_demux, ts_pid_t *pid, block_t *p_pkt, bool ...@@ -1587,7 +1587,7 @@ static bool PushPESBlock( demux_t *p_demux, ts_pid_t *pid, block_t *p_pkt, bool
p_pes->gather.p_data = NULL; p_pes->gather.p_data = NULL;
p_pes->gather.i_data_size = 0; p_pes->gather.i_data_size = 0;
p_pes->gather.i_gathered = 0; p_pes->gather.i_gathered = 0;
p_pes->gather.pp_last = &pid->u.p_pes->gather.p_data; p_pes->gather.pp_last = &p_pes->gather.p_data;
ParsePESDataChain( p_demux, pid, p_datachain ); ParsePESDataChain( p_demux, pid, p_datachain );
b_ret = true; b_ret = true;
} }
...@@ -1599,6 +1599,7 @@ static bool PushPESBlock( demux_t *p_demux, ts_pid_t *pid, block_t *p_pkt, bool ...@@ -1599,6 +1599,7 @@ static bool PushPESBlock( demux_t *p_demux, ts_pid_t *pid, block_t *p_pkt, bool
{ {
/* msg_Dbg( p_demux, "broken packet" ); */ /* msg_Dbg( p_demux, "broken packet" ); */
block_Release( p_pkt ); block_Release( p_pkt );
return b_ret;
} }
block_ChainLastAppend( &p_pes->gather.pp_last, p_pkt ); block_ChainLastAppend( &p_pes->gather.pp_last, p_pkt );
......
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