- 23 Feb, 2016 1 commit
-
-
Anton Khirnov authored
Currently, AVStream contains an embedded AVCodecContext instance, which is used by demuxers to export stream parameters to the caller and by muxers to receive stream parameters from the caller. It is also used internally as the codec context that is passed to parsers. In addition, it is also widely used by the callers as the decoding (when demuxer) or encoding (when muxing) context, though this has been officially discouraged since Libav 11. There are multiple important problems with this approach: - the fields in AVCodecContext are in general one of * stream parameters * codec options * codec state However, it's not clear which ones are which. It is consequently unclear which fields are a demuxer allowed to set or a muxer allowed to read. This leads to erratic behaviour depending on whether decoding or encoding is being performed or not (and whether it uses the AVStream embedded codec context). - various synchronization issues arising from the fact that the same context is used by several different APIs (muxers/demuxers, parsers, bitstream filters and encoders/decoders) simultaneously, with there being no clear rules for who can modify what and the different processes being typically delayed with respect to each other. - avformat_find_stream_info() making it necessary to support opening and closing a single codec context multiple times, thus complicating the semantics of freeing various allocated objects in the codec context. Those problems are resolved by replacing the AVStream embedded codec context with a newly added AVCodecParameters instance, which stores only the stream parameters exported by the demuxers or read by the muxers.
-
- 02 Jan, 2016 1 commit
-
-
Andreas Cadhalpun authored
Also use a unsigned constant for the shift calculation, as 1 << 31 is undefined for int32_t. This is also fixed oggparsetheora. This fixes ubsan runtime error: shift exponent is too large for 32-bit type 'int' Reviewed-by: Michael Niedermayer <michael@niedermayer.cc> Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
-
- 04 Dec, 2015 1 commit
-
-
Michael Niedermayer authored
Fixes: 1536b9b096a8f95b742bae9d3d761cc6/signal_sigsegv_294aaed_7322_4fad88a38dc8952dc20dcb60c1895758.ogg Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-
- 27 Jul, 2015 1 commit
-
-
Vittorio Giovara authored
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
-
- 13 Aug, 2014 1 commit
-
-
Andrew Stone authored
Originally, AVFormatContext and a metadata dict were provided to ff_vorbis_comment(), but this presented issues if an AVStream was being updated or the metadata on AVFormatContext wasn't actually being updated. To remedy this, ff_vorbis_stream_comment() explicitly updates a stream's metadata and sets any necessary flags. ff_vorbis_comment() does not modify any flags, and any calls to it that update AVFormatContext's metadata (just a single call) must also update AVFormatContext.event_flags after detecting any metadata changes to the provided dictionary, as signaled by a positive return value. Signed-off-by: Anton Khirnov <anton@khirnov.net>
-
- 28 May, 2014 1 commit
-
-
Anton Khirnov authored
It is commonly stored in a vorbiscomment block in codec private data.
-
- 15 Dec, 2013 1 commit
-
-
Michael Niedermayer authored
Fixes use of uninitialized memory Fixes: msan_uninit-mem_7f66fa0ac09d_9197_theora_a4_v6_k250_s0.ogg Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-
- 31 Oct, 2013 1 commit
-
-
Anton Khirnov authored
-
- 30 Oct, 2013 3 commits
-
-
Anton Khirnov authored
-
Anton Khirnov authored
-
Anton Khirnov authored
Also typedef the private data struct and make its name consistent with the rest of Libav. Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com> Signed-off-by: Anton Khirnov <anton@khirnov.net>
-
- 26 Sep, 2013 1 commit
-
-
Martin Storsjö authored
When av_reallocp fails, the associated variables that keep track of the number of elements in the array (and in some cases, the separate number of allocated elements) need to be reset. Not all of these might technically be needed, but it's better to reset them if in doubt, to make sure variables don't end up conflicting. Signed-off-by: Martin Storsjö <martin@martin.st>
-
- 18 Sep, 2013 1 commit
-
-
Alexandra Khirnova authored
Signed-off-by: Diego Biurrun <diego@biurrun.de>
-
- 14 Jan, 2013 1 commit
-
-
Giorgio Vazzana authored
Pass the correct header size to ff_vorbis_comment() Signed-off-by: Martin Storsjö <martin@martin.st>
-
- 05 Oct, 2012 1 commit
-
-
Giorgio Vazzana authored
Pass the correct header size to ff_vorbis_comment() Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-
- 25 Sep, 2012 1 commit
-
-
Michael Niedermayer authored
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-
- 24 Sep, 2012 3 commits
-
-
Luca Barbato authored
Not having the header for a codec is a tell-tale of a broken file.
-
Luca Barbato authored
-
Michael Niedermayer authored
Additional safety in case a special ogg stream is crafted with the proper number of Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-
- 12 Aug, 2012 1 commit
-
-
Michael Niedermayer authored
Fixes Ticket1508 Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-
- 07 Aug, 2012 1 commit
-
-
Anton Khirnov authored
-
- 04 May, 2012 2 commits
-
-
Michael Niedermayer authored
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
code based on the solution in vorbis Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-
- 23 Mar, 2012 1 commit
-
-
Michael Niedermayer authored
This prevents a null ptr dereference. It could be checked differently but this way it should be possible to return some data. Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-
- 02 Mar, 2012 1 commit
-
-
Anton Khirnov authored
Demuxers are not supposed to set it.
-
- 30 Nov, 2011 1 commit
-
-
Anton Khirnov authored
It's supposed to be called only from (de)muxers.
-
- 06 Feb, 2011 1 commit
-
-
Reimar Döffinger authored
Signed-off-by: Ronald S. Bultje <rsbultje@gmail.com>
-
- 05 Feb, 2011 1 commit
-
-
Reimar Döffinger authored
-
- 31 Mar, 2010 1 commit
-
-
Jean-Daniel Dupas authored
Patch by Jean-Daniel Dupas, devlists shadowlab org Originally committed as revision 22744 to svn://svn.ffmpeg.org/ffmpeg/trunk
-
- 30 Mar, 2010 1 commit
-
-
Stefano Sabatini authored
is deprecated and will be dropped at the next major bump. Originally committed as revision 22735 to svn://svn.ffmpeg.org/ffmpeg/trunk
-
- 12 Mar, 2010 2 commits
-
-
David Conrad authored
Originally committed as revision 22473 to svn://svn.ffmpeg.org/ffmpeg/trunk
-
David Conrad authored
Originally committed as revision 22472 to svn://svn.ffmpeg.org/ffmpeg/trunk
-
- 05 Mar, 2010 1 commit
-
-
David Conrad authored
Fixes issue746 Originally committed as revision 22214 to svn://svn.ffmpeg.org/ffmpeg/trunk
-
- 11 Jan, 2010 1 commit
-
-
David Conrad authored
Originally committed as revision 21134 to svn://svn.ffmpeg.org/ffmpeg/trunk
-
- 03 Jul, 2009 1 commit
-
-
Reimar Döffinger authored
Avoids division by 0 due to e.g. time_base denominator being 0. Originally committed as revision 19334 to svn://svn.ffmpeg.org/ffmpeg/trunk
-
- 05 Jun, 2009 1 commit
-
-
David Conrad authored
Spotted by valgrind. Originally committed as revision 19119 to svn://svn.ffmpeg.org/ffmpeg/trunk
-
- 14 May, 2009 1 commit
-
-
David Conrad authored
This fixes ogg/theora on ARM (more generally the A32 bitstream reader) Originally committed as revision 18819 to svn://svn.ffmpeg.org/ffmpeg/trunk
-
- 13 Apr, 2009 1 commit
-
-
Stefano Sabatini authored
Originally committed as revision 18494 to svn://svn.ffmpeg.org/ffmpeg/trunk
-
- 06 Nov, 2008 2 commits
-
-
Måns Rullgård authored
Originally committed as revision 15785 to svn://svn.ffmpeg.org/ffmpeg/trunk
-
Måns Rullgård authored
Originally committed as revision 15784 to svn://svn.ffmpeg.org/ffmpeg/trunk
-