Fixes ticket1321
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Fixes out of array reads
Fixes Ticket1330
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Fixes Ticket1241
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NPaul B Mahol <onemda@gmail.com>

The assert can be false with some invalid inputs, the check is
too expensive to always do though for just a warning message.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

It is possible in various error pathes as well as gap handling
that this has already been allocated. Its not clear why that
would be a problem with the current code, thus disable the
assert to avoid common assert failure when asserts are enabled.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Ensures that the next frame decoded after seeking will be decoded as a
keyframe.

Fixes Ticket1273
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

This assert is no longer true since h264 error concealment needs
last_picture_ptr to be set.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

it now just checks uninitialized and unused data.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

CC: libav-stable@libav.org

The check is bogus since the nuv frameheader is already skipped
and the (decompressed) RTjpeg header is checked.

This reverts commit f6afacdb.

CC: libav-stable@libav.org

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

IMCContext was changed from an array to a pointer in 66b84e4a,
but this memset() was not updated.
Signed-off-by: NMans Rullgard <mans@mansr.com>

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

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

They are available in mpc.h
Signed-off-by: NPaul B Mahol <onemda@gmail.com>

Signed-off-by: NPaul B Mahol <onemda@gmail.com>

Some corrupted files would end up with a negative offset,
and segfault.

Fixes bug #177.
Signed-off-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

We're now running some of this code through valgrind for the first
time, and a few warnings showed up stemming from two problems.

1) The ASS code assumes the subtitle header is null terminated, but
it wasn't, and passing the size down doesn't look like fun, so I
added a terminator

2) The code wasn't freeing all of its state.
Signed-off-by: NPhilip Langdale <philipl@overt.org>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Based on a patch by Piotr Bandurski.
Signed-off-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

This is a reference that leaked in from my future work to
support styling.
Signed-off-by: NPhilip Langdale <philipl@overt.org>

If there was a failure inflating, or reinitializing
the zstream, the current frame's buffer would be lost.
Signed-off-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

Be consistent with error messages and code formatting.
Signed-off-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

If there was a failure inflating, or reinitializing
the zstream, the current frame's buffer would be lost.
Signed-off-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

Be consistent with error messages and code formatting.
Signed-off-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

This change introduces a basic encoder for 3GPP Timed Text subtitles,
also known as TX3G, Quicktime subtitles, or "movtext" in the existing
code.

This initial change doesn't attempt to write styling information,
and just writes the plain text of the subtitles. I intend to add
support for styles eventually, but it's challenging due to a lack
of existing players that support them.

Note that an additional change is required to the mov/mp4 muxer to
write empty subtitle packets to indicate subtitle duration.
Signed-off-by: NPhilip Langdale <philipl@overt.org>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

In the GNU assembler, a relational expression, bizarrely, has the
value -1 if true, whereas in Apple's it is +1.  This patch makes
sure the correct expression is used in both cases.
Signed-off-by: NMans Rullgard <mans@mansr.com>

The clang integrated assembler does not support pre-UAL syntax,
while gcc requires pre-UAL syntax for ARM code.  A patch[1] for
clang to support the old syntax as well has been ignored since
January.

This patch chooses the syntax appropriate for each compiler,
allowing both to build the code.  Notably, this change allows
building for iphone with the latest Apple Xcode update.

[1] http://llvm.org/bugs/show_bug.cgi?id=11855Signed-off-by: NMans Rullgard <mans@mansr.com>

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

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>