Progressive data is allocated later in decode_sof(), not allocating
that data leads to NULL dereferences.

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org

This feature is complex, of questionable utility, and slows down
normal decoding.

Signed-off-by: Mans Rullgard <mans@mansr.com>

Also remove one pointless zero initialization in rangecoder.c.

libavcodec/mjpegdec.c:1463: warning: format ‘%x’ expects type ‘unsigned int’, but argument 5 has type ‘const uint8_t *’

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org

Signed-off-by: Martin Storsjö <martin@martin.st>

Signed-off-by: Martin Storsjö <martin@martin.st>

also check for allocation failure

Signed-off-by: Justin Ruggles <justin.ruggles@gmail.com>

Signed-off-by: Diego Biurrun <diego@biurrun.de>

Check explicitly if enough bits are left to prevent an infinite loop
when the bitstream buffer is not followed by zero-padding.

Based on patches by Michael Niedermayer <michaelni@gmx.at>.

The 'fiel' atoms can be found in H.264 tracks clobbering the extradata.
MJPEG supports non field based extradata, and this data should be
preserved when copying.

Signed-off-by: Anton Khirnov <anton@khirnov.net>

Deprecate CODEC_FLAG_EXTERN_HUFF

Signed-off-by: Anton Khirnov <anton@khirnov.net>

Signed-off-by: Ronald S. Bultje <rsbultje@gmail.com>

It's more readable and less prone to breakage.

Fixes decoding of MJPEG files produced by some UVC Logitec web cameras,
such as "Notebook Pro" and "HD C910".

References:
http://trac.videolan.org/vlc/ticket/4215
http://ffmpeg.org/trac/ffmpeg/ticket/267

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
Reviewed-by: Kostya <kostya.shishkov@gmail.com>
(cherry picked from commit 7b8ed831eb)

This removes a number of compiler warnings.

Signed-off-by: Mans Rullgard <mans@mansr.com>

Signed-off-by: Mans Rullgard <mans@mansr.com>

Signed-off-by: Diego Biurrun <diego@biurrun.de>

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
Signed-off-by: Ronald S. Bultje <rbultje@google.com>

Signed-off-by: Anton Khirnov <anton@khirnov.net>

Signed-off-by: Anton Khirnov <anton@khirnov.net>

Signed-off-by: Anton Khirnov <anton@khirnov.net>

Signed-off-by: Mans Rullgard <mans@mansr.com>

While 4 works for some samples, 3 is the correct value since 8x8
DCT is used by (m)jpeg.

Signed-off-by: Janne Grunau <janne-ffmpeg@jannau.net>

It is pretty hopeless that other considerable projects will adopt
libavutil alone in other projects. Projects that need small footprint
are better off with more specialized libraries such as gnulib or rather
just copy the necessary parts that they need. With this in mind, nobody
is helped by having libavutil and libavcore split. In order to ease
maintenance inside and around FFmpeg and to reduce confusion where to
put common code, avcore's functionality is merged (back) to avutil.

Signed-off-by: Reinhard Tartler <siretart@tauware.de>

dprintf clashes with POSIX.1-2008

None of these symbols should be accessed directly, so declare them as
hidden.

Signed-off-by: Mans Rullgard <mans@mansr.com>

Some of the macros in get_bits.h include a final semicolon,
some do not.  This removes these or adds do {} while(0) around
the macros as appropriate and adds semicolons where needed in
calling code.

Signed-off-by: Mans Rullgard <mans@mansr.com>

Originally committed as revision 26386 to svn://svn.ffmpeg.org/ffmpeg/trunk

Originally committed as revision 25549 to svn://svn.ffmpeg.org/ffmpeg/trunk

Originally committed as revision 25548 to svn://svn.ffmpeg.org/ffmpeg/trunk

Originally committed as revision 25547 to svn://svn.ffmpeg.org/ffmpeg/trunk

This fixes a possibly exploitable buffer overflow and it will likely also be needed for future overreading fixes.

Originally committed as revision 25546 to svn://svn.ffmpeg.org/ffmpeg/trunk

Originally committed as revision 25545 to svn://svn.ffmpeg.org/ffmpeg/trunk

Originally committed as revision 25544 to svn://svn.ffmpeg.org/ffmpeg/trunk

Originally committed as revision 25543 to svn://svn.ffmpeg.org/ffmpeg/trunk

Originally committed as revision 25542 to svn://svn.ffmpeg.org/ffmpeg/trunk

Originally committed as revision 25541 to svn://svn.ffmpeg.org/ffmpeg/trunk

Originally committed as revision 25540 to svn://svn.ffmpeg.org/ffmpeg/trunk

Originally committed as revision 25058 to svn://svn.ffmpeg.org/ffmpeg/trunk

Because the order of evaluation of subexpressions is undefined, two
get_bits() calls may not be part of the same expression.

See also r24902.

Originally committed as revision 24906 to svn://svn.ffmpeg.org/ffmpeg/trunk

av_check_image_size() instead.

Originally committed as revision 24711 to svn://svn.ffmpeg.org/ffmpeg/trunk

Originally committed as revision 24170 to svn://svn.ffmpeg.org/ffmpeg/trunk