This function was broken when the start bin was not at the start
of a band.
Signed-off-by: NMans Rullgard <mans@mansr.com>

Also prettyprint some comments in Assembly code.

This is more consistent with all the other codec names.
Signed-off-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NDiego Biurrun <diego@biurrun.de>

Do not use AVStream's duration for dts generation since it contains in
some cases the duration of the whole file instead of duration of the
samples in the moov. This happens if the mdhd holds the duration of the
whole file but has no entries or a zero duration in its stts.

There is no point in this test using the RM format.
Signed-off-by: NMans Rullgard <mans@mansr.com>

Signed-off-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NJustin Ruggles <justin.ruggles@gmail.com>

Return the correct number of consumed bytes and set *data_size = 0.
Returned size is 1 too small, leading to that 1 byte being read as the next
frame, which results in an extra blank frame at the beginning of the stream.

Avoids doing malloc/free for each frame.
Also fixes valgrind errors due to use of uninitialized padding bytes.

Based on a patch by Reimar Döffinger <Reimar.Doeffinger@gmx.de>

Signed-off-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NJustin Ruggles <justin.ruggles@gmail.com>

Specifies the correct number of xmm registers used so that they can be saved
and restored on Win64 if necessary.

It is already the correct size as set by ff_alloc_packet().

This will simplify encoders which use this function to request the exact
packet size rather than the maximum size.

Signed-off-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NJustin Ruggles <justin.ruggles@gmail.com>

Ensures alignment and avoids using uninitialized data.
Signed-off-by: NJustin Ruggles <justin.ruggles@gmail.com>

also check for allocation failure
Signed-off-by: NJustin Ruggles <justin.ruggles@gmail.com>

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

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

Wrapper around av_fast_malloc() that keeps FF_INPUT_BUFFER_PADDING_SIZE
zero-padded bytes at the end of the used buffer.

Based on a patch by Reimar Döffinger <Reimar.Doeffinger@gmx.de>.

This was accidently disabled.
Signed-off-by: NRonald S. Bultje <rsbultje@gmail.com>

Also use av_log_ask_for_sample() where it makes sense.
Signed-off-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NDiego Biurrun <diego@biurrun.de>

Signed-off-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NDiego Biurrun <diego@biurrun.de>

CC: libav-stable@libav.org

DSPContext.bswap_buf() requires aligned output
Signed-off-by: NRonald S. Bultje <rsbultje@gmail.com>

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: NAlex Converse <alex.converse@gmail.com>

Found with asan.

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: NAlex Converse <alex.converse@gmail.com>

dv: Fix null pointer dereference due to ach=0

Fixes part2 of CVE-2011-3929

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Reviewed-by: NRoman Shaposhnik <roman@shaposhnik.org>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>
Signed-off-by: NAlex Converse <alex.converse@gmail.com>

dv: check stype

Fixes part1 of CVE-2011-3929
Possibly fixes part of CVE-2011-3936

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Reviewed-by: NRoman Shaposhnik <roman@shaposhnik.org>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>
Signed-off-by: NAlex Converse <alex.converse@gmail.com>

get_ue_golomb_long() is only tested for values up to 2^15 - 2 since
we can not write larger values.
Silence the test on success and return a non-zero value on error.
Use an heap scratch buffer instead of large stack buffer.
Remove unneeded includes.

The new name is more appropriate as only golomb functions are tested.