Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: NRonald S. Bultje <rsbultje@gmail.com>

Also remove some write-only variables or write-only variable
assignments, remove internal colorspace conversion to native
endianness (that can be done by swscale much more efficiently),
and some cosmetics.

Prevents running error resilience on a previous frame which will write
to the pic->mb_type[] array of the previous image. The array might
already be re-used for a new image in a subsequent thread, thus cause
two threads to write to the same pic->mb_type[] array, causing a race
condition which can crash in rv34_decode_cbp(), called by
rv34_decode_inter_mb_header() (which accesses mb_type[] twice,
assuming values are maintained, which the race condition breaks).

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

This allows simplifying the Makefiles; it is no longer necessary to register
arch subdirectory Makefiles, just putting them in place is enough.

Remove confusing sentence that implied the user should set the timebase.
Elaborate on how the timebase is set for muxing.

Signed-off-by: NJindrich Makovicka <jindrich.makovicka@nangu.tv>
Signed-off-by: NMartin Storsjö <martin@martin.st>

This makes them safe to use in non-fully braced if-blocks and similar.

This isn't exactly equivalent with the earlier code for codecs
other than H264 and VC1, but those are two only codecs supported
by this codepath anyway, and it simplifies it a bit.
Signed-off-by: NMartin Storsjö <martin@martin.st>

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

Before this, they were only added to the delayed release queue and not
freed until later. This could lead to unnecessary memory use or buffer
exhaustion.
Signed-off-by: NRonald S. Bultje <rsbultje@gmail.com>

Fixes a crash when FF_DEBUG_PICT_INFO is used.
Signed-off-by: NRonald S. Bultje <rsbultje@gmail.com>

Fixes crash during slice size reading if slice_end goes negative.

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

Make movenc use this function instead of the current custom
conversion function.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Else we run out of progress variables after a few failed buffer
allocations.
Signed-off-by: NAnton Khirnov <anton@khirnov.net>

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

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

Spotted by Антон.

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

lf_delta.ref[i] and lf_delta.mode[i] were incorrectly reset to 0 if
specific delta value was not updated. Fixed to keep the previous value
if flag indicates that element in question is not updated.
Signed-off-by: NJanne Salonen <jsalonen@google.com>
Signed-off-by: NRonald S. Bultje <rsbultje@gmail.com>