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

Instead print an error and continue

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

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

This is currently not supported
Fixes part of Ticket 3539
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Fixes reading from freed data
Fixes part of Ticket3539
Reviewed-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NJames Almer <jamrial@gmail.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Fixes use of uninitialized memory
Fixes part of msan_uninit-mem_7f055dd0ab1b_9558_videopop_guitar_300k.swf
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Fixes use of uninitialized memory
Fixes part of msan_uninit-mem_7f055dd0ab1b_9558_videopop_guitar_300k.swf
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Fixes use of uninitialized memory
Fixes msan_uninit-mem_7f90d9cce964_9558_videopop_guitar_300k.swf
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

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

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

Implemented since 624fb5f9.

This also makes the changes of a3949fe1 applicable in both cases.

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

Avoids trying to read a packet with 0 or negative size.
Avoids a potential infinite loop due to seeking backwards.

Partially based on a patch by Michael Niedermayer.

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

Reviewed-by: Clément Bœsch

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

We now only return in the middle of the refill in case of read error, so
inflate can be re-called if zlib needs an empty (z.avail_in=0) inflate
call for flushing.

Signed-off-by: NMichael Bradshaw <mbradshaw@sorensonmedia.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

It is not supposed to be set outside of lavc.

Fixes a divide by zero when the stored framerate is 0.

Prevents crashers when using the packet if allocation failed.

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

It's supposed to be called only from (de)muxers.

Manual replacements are done in this commit.

In many cases, the id is some constant made up number (e.g. 0 for video
and 1 for audio), which is then not used in the demuxer for anything.
Those ids are removed.

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