- 29 Apr, 2015 22 commits
-
-
Michael Niedermayer authored
* commit '440e8dd3': h264: drop a comment that carries no useful information Merged-by:
Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
* commit '11f024ef': h264: move freeing the escaped RBSP buffer to free_context() Merged-by:
Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
* commit 'ee62b364': h264: eliminate ff_h264_set_parameter_from_sps() Conflicts: libavcodec/h264.c libavcodec/h264.h libavcodec/h264_slice.c Merged-by:
Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
* commit '54986d6d': h264: move context reinit lower down in update_thread_context() Conflicts: libavcodec/h264_slice.c Merged-by:
Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
* commit 'a6cd1544': h264: move the DPB init/uninit to init_context()/free_context() Conflicts: libavcodec/h264.c Merged-by:
Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
* commit '9d33bab5': h264: drop H264Context.ouputed_poc Conflicts: libavcodec/h264.c Merged-by:
Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
* commit '27b0e6eb': h264: drop needs_realloc Merged-by:
Michael Niedermayer <michaelni@gmx.at>
-
Anton Khirnov authored
-
Anton Khirnov authored
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.
-
Anton Khirnov authored
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.
-
Anton Khirnov authored
It uses some fields from the SPS, which is not yet set where the reinit is called currently.
-
Anton Khirnov authored
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.
-
Anton Khirnov authored
It is only set, but never used for anything.
-
Anton Khirnov authored
It is not needed anymore since switching to refcounted frames.
-
Michael Niedermayer authored
* commit '0ac8ff61': avresample: Reallocate the internal buffer to the correct size Merged-by:
Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
Signed-off-by:
Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
* commit '82de8d71': mpegts: Update the PSI/SI table only if the version change Conflicts: libavformat/mpegts.c Merged-by:
Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
* commit '0c69164f': h263: Convert function to macro Conflicts: libavcodec/h263.h Merged-by:
Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
* commit '59dfc2b0': h263: Drop commented-out code and a related otherwise unused function Merged-by:
Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
* commit 'c3418201': jpeglsenc: Mark codec as init-thread-safe and init-cleanup Merged-by:
Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
* commit '3919a457': jpeglsenc: Check memory allocations Conflicts: libavcodec/jpeglsenc.c See: 48214956Merged-by:
Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
* commit 'f5ba67ee': flacenc: Move a scratch buffer to struct used by the function Conflicts: libavcodec/flacenc.c Merged-by:
Michael Niedermayer <michaelni@gmx.at>
-
- 28 Apr, 2015 18 commits
-
-
Luca Barbato authored
Fixes the corner case in which the internal buffer size is larger than input buffer provided and resizing it before moving the left over samples would make it write to now unallocated memory. Bug-Id: 825 CC: libav-stable@libav.org Signed-off-by:
Luca Barbato <lu_zero@gentoo.org>
-
Andreas Cadhalpun authored
The existing check has two problems: 1) i + count can overflow, so that the check '< 256' returns true. 2) In the (i == 'N') case occurs a j-- so that the loop runs once more. This can trigger the assertion 'nut->header_len[0] == 0' or cause segmentation faults or infinite hangs. Signed-off-by:
Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> Signed-off-by:
Michael Niedermayer <michaelni@gmx.at>
-
Andreas Cadhalpun authored
Signed-off-by:
Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> Signed-off-by:
Michael Niedermayer <michaelni@gmx.at>
-
Andreas Cadhalpun authored
This fixes a segmentation fault when accessing the metadata. Signed-off-by:
Michael Niedermayer <michaelni@gmx.at>
-
John Högberg authored
If a PAT is finished while a PMT section filter is opened but not yet finished, the PMT section filter is closed and all the received data is discarded. This is usually not an issue but some multiplexers (With very quick PAT/PMT repetition settings) consistently emit a PMT section start, then a PAT, and then the rest of the PMT, causing the aforementioned behavior to result in no PMT being finished. In the most pathologic situation the stream information are lost and the probe fallback miscategorizes subtitles as mp3 audio. Avoid the issue through eliminating redundant PSI/SI table updates by checking their version field, which is required by the standard to be incremented on every change no matter how minor. CC: libav-stable@libav.org Signed-off-by:
Luca Barbato <lu_zero@gentoo.org>
-
Andreas Cadhalpun authored
A negative frame rate triggers an av_assert2 in av_rescale_rnd. Signed-off-by:
Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> Signed-off-by:
Michael Niedermayer <michaelni@gmx.at>
-
Michael Niedermayer authored
Signed-off-by:
Michael Niedermayer <michaelni@gmx.at>
-
foo86 authored
Check extended sync word for 16-bit LE and BE core streams to reduce probability of alias sync detection. Previously sync word extension was checked only for 14-bit streams. This follows up the similar change in avcodec/dca_parser. Signed-off-by:
Michael Niedermayer <michaelni@gmx.at>
-
Vittorio Giovara authored
-
Vittorio Giovara authored
-
Vittorio Giovara authored
Fixes a memory leak on init failure.
-
Vittorio Giovara authored
Convert exisiting free functions to av_freep() to avoid accidental double frees, and always intialize all buffers to NULL.
-
Vittorio Giovara authored
This avoids allocating/freeing memory at every function call, checking its return value, and carrying the error around.
-
Michael Niedermayer authored
Signed-off-by:
Michael Niedermayer <michaelni@gmx.at>
-
Andreas Cadhalpun authored
s->decoded_buffer is allocated with a min_size of: 2 * FFALIGN(blockstodecode, 8) * sizeof(*s->decoded_buffer) Then it is assigned to s->decoded[0] (and s->decoded_buffer + FFALIGN(blockstodecode, 8) to s->decoded[1]) and passed as out buffer to decode_array_0000. In this function 64 elements of the out buffer are written unconditionally and outside the array if blockstodecode is too small. This causes memory corruption, leading to segmentation faults or other crashes. Thus change decode_array_0000 to write at most blockstodecode elements of the out buffer. Signed-off-by:
Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> Signed-off-by:
Michael Niedermayer <michaelni@gmx.at>
-
Maneesh Gupta authored
AVOpenCLDeviceNode and AVOpenCLPlatformNode used fixed static buffer for holding the device and platform name. This patch modifies these structures to use pointers instead. The memory required to hold the names is now dynamically allocated, the size for which is determined by querying appropriate OpenCL runtime APIs. Signed-off-by:
Maneesh Gupta <maneesh.gupta@amd.com> Signed-off-by:
Michael Niedermayer <michaelni@gmx.at>
-
Maneesh Gupta authored
The opt_opencl_bench function copied the device name using strcpy without checking if the source string was larger. This patch fixes this by replacing the strcpy with av_strlcpy, with the string copy size capped to the destination buffer size. Signed-off-by:
Maneesh Gupta <maneesh.gupta@amd.com> Signed-off-by:
Michael Niedermayer <michaelni@gmx.at>
-
Vignesh Venkatasubramanian authored
Remove the direct profile from UTCTiming element. Per DASH spec, direct profile value should be the time at which the request was made to the server and not the time at which the manifest was written. So ffmpeg cannot write this value. This patch removes the direct profile and write the UTCTiming element with the http profile only if a URL is passed as a parameter. Update the fate test to reflect this change. Signed-off-by:
Vignesh Venkatasubramanian <vigneshv@google.com> Signed-off-by:
Michael Niedermayer <michaelni@gmx.at>
-