[vlc-devel] vlc: svn commit r22339 (fenrir)

[Rémi Denis-Courmont]

Le Monday 01 October 2007 01:46:04 Laurent Aimar, vous avez écrit :
> On Mon, Oct 01, 2007, Subversion daemon wrote:
> >  I see it calling demux2_Control, this will segfault if you are unlucky.
>
>  I will add a input_Control for the get fps (and remove the
> demux2_Control).
>
>  But can someone have a look how to remove the access to p_input->p ?

Same problem here:

./stream_out/transcode.c:            stats_UpdateInteger( p_input, 
p_input->p->counters.p_decoded_audio,
./stream_out/transcode.c:            stats_UpdateInteger( p_input, 
p_input->p->counters.p_decoded_video,
./control/http/http.c:        if( p_sys->p_input && p_sys->p_input->p )
./control/http/mvar.c:    if( p_input == NULL || p_input->p == NULL /* 
workarround assert in input_GetItem */ )
./control/netsync.c:    if( !p_input || !p_input->p->p_es_out ) return 0;
./control/netsync.c:    if( es_out_Control( p_input->p->p_es_out, 
ES_OUT_GET_TS,

Any ideas?

(Side note: I especially like the insult in the HTTP interface against 
zorglug's assert from the one committer who regurlarly criticize my rough 
commit log style, but that's off-topic)

-- 
Rémi Denis-Courmont
http://www.remlab.net/