| Commit message (Collapse) | Author | Age |
|
|
|
|
|
|
|
|
| |
avctx->bits_per_raw_sample is used in get_sbits_long, which only
supports up to 32 bits.
CC: libav-stable@libav.org
Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>
|
|
|
|
|
|
| |
CC: libav-stable@libav.org
Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>
|
|
|
|
|
|
| |
Regression introduced in 598f7d046cbf306706623210c5baafa3b7cd1df3.
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
|
|
|
|
|
|
| |
warning: assigning to 'AVFrame *' (aka 'struct AVFrame *') from
'const AVFrame *' (aka 'const struct AVFrame *') discards
qualifiers [-Wincompatible-pointer-types-discards-qualifiers]
|
| |
|
| |
|
|
|
|
|
| |
It now has a fixed size and is initialized only once, so there is no
point in allocating it separately.
|
|
|
|
|
| |
There is no real need to handle the init case specially, everything
necessary is initialized in the reinit code as well.
|
|
|
|
|
| |
It is only used to decide whether to call free_tables(), but that
function is safe to call on an uninitialized context as well.
|
|
|
|
|
| |
With frame threading, it is currently only updated in the context where
the change occurs, but not in any other contexts.
|
|
|
|
|
|
| |
It does not make sense to copy is_avc without copying this as well. This
patch should not change anything for now, but will have an effect in
later commits.
|
| |
|
|
|
|
|
| |
It does not logically belong in free_tables(), since it's not allocated
in alloc_tables() and its size has nothing to do with the frame size.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
That function currently does two things -- reinitializing the DSP
contexts and setting low_delay based on the SPS values.
The former more appropriately belongs in h264_slice_header_init(), while
the latter only really makes sense in decode_slice_header().
The third call to ff_h264_set_parameter_from_sps(), done immediately
after parsing a new SPS, appears to serve no useful purpose, so it is
just dropped.
Also, drop now unneeded H264Context.cur_chroma_format_idc.
|
|
|
|
|
| |
It uses some fields from the SPS, which is not yet set where the reinit
is called currently.
|
|
|
|
|
|
|
|
|
|
| |
Currently, the DPB is initialized in alloc_tables() and uninitialized in
free_tables(), but those functions manage frame size-dependent
variables, so DPB management does not logically belong in there.
Since we want the init/uninit to happen exactly once per the context
lifetime, init_context()/free_context() are the proper place for this
code.
|
|
|
|
| |
It is only set, but never used for anything.
|
|
|
|
| |
It is not needed anymore since switching to refcounted frames.
|
| |
|
| |
|
|
|
|
| |
Fixes a memory leak on init failure.
|
|
|
|
|
| |
Convert exisiting free functions to av_freep() to avoid accidental
double frees, and always intialize all buffers to NULL.
|
|
|
|
|
| |
This avoids allocating/freeing memory at every function call,
checking its return value, and carrying the error around.
|
|
|
|
| |
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
|
|
|
|
| |
Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
|
|
|
|
| |
This takes care of memory leaks on init error.
|
|
|
|
| |
This takes care of memory leaks on init error.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
The table had a wrong table height and was missing the first line of bytes.
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
|
|
|
|
| |
Improves Coverity analysis, avoiding "double semicolon" CIDs.
|
|
|
|
| |
Improve the compatibility with other software.
|
|
|
|
|
| |
The `hq_ac_vlc.table` is incomplete, so unaccounted symbol return -1
as value.
|
| |
|
| |
|
|
|
|
| |
Value is stored as a byte.
|
|
|
|
| |
It is required by ptrdiff_t, fixes make checkheaders.
|
|
|
|
| |
Bug-Id: CID 1026763
|
|
|
|
| |
Bug-Id: CID 1295124
|
|
|
|
| |
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
|
|
|
|
| |
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
|
|
|
|
|
|
| |
ac may be NULL and then accessing ac->avctx results in a segmentation fault.
Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
|
| |
|
|
|
|
|
|
|
|
| |
warning: incompatible
pointer types assigning to 'me_cmp_func' (aka 'int (*)(struct
MpegEncContext *, uint8_t *, uint8_t *, ptrdiff_t, int)') from 'int
(MpegEncContext *, uint8_t *, uint8_t *, int, int)'
[-Wincompatible-pointer-types]
|
|
|
|
| |
Fix a rather lengthy initialization warning from clang.
|
|
|
|
|
|
| |
Based on work by Kostya Shishkov <kostya.shishkov@gmail.com>.
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
|
|
|
|
|
| |
Add some bounds checking to CLLC; reduce HQX variable scoping,
add an error message.
|
|
|
|
|
|
| |
Inspired by a patch from Ferdinand Oeinck <ferdo@demon.nl>.
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
|