[FFmpeg-devel] [PATCH]Fix MPlayer crash in decode_unregistered_user_data()

Michael Niedermayer michaelni
Mon Aug 10 13:19:53 CEST 2009


On Mon, Aug 10, 2009 at 08:39:27AM +0000, Carl Eugen Hoyos wrote:
> Michael Niedermayer <michaelni <at> gmx.at> writes:
> 
> > > > why is avctx NULL ?
> > > 
> > > Currently, the following is done:
> > > 
> > >     avcodec_register_all();
> > >     AVCodecParserContext * h264parserctx = av_parser_init(CODEC_ID_H264);
> > > ...
> > >     h264parserctx->parser->parser_parse(h264parserctx, NULL...
> > > 
> > > Is the correct way to use parser_parse to also call
> > >     AVCodecContext * parser_context = avcodec_alloc_context
> > > at initialisation and pass this context whenever calling the parser?
> > 
> > is there a disadvantage in allocating a AVCodecContext?
> 
> You mean allocate it without using avcodec_alloc_context? I thought that
> wouldn't work well with a shared libavcodec library.

no, i meant
is there any point in passing NULL instead of allocating it and passing that?

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When the tyrant has disposed of foreign enemies by conquest or treaty, and
there is nothing more to fear from them, then he is always stirring up
some war or other, in order that the people may require a leader. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090810/c9c1f2c5/attachment.pgp>



More information about the ffmpeg-devel mailing list