FFmpeg

Commit Graph

Author	SHA1	Message	Date
Sean McGovern	cb167f2947	h264_refs: validate the SPS pointer in ff_h264_execute_ref_pic_marking() Bug-Id: 1036 CC: libav-stable@libav.org	8 years ago
Diego Biurrun	d4c2103bd3	golomb: Convert to the new bitstream reader	8 years ago
Diego Biurrun	b96f0ab3d2	h264: Kill broken disabled cruft	8 years ago
Anton Khirnov	b13fc1e344	h264: do not pass H264Context to h264_slice_header_parse() This should make it more clear that this function does not need any decoder-global state other than the parameter sets.	9 years ago
Anton Khirnov	5c2fb561d9	h264: add H264_ prefix to the NAL unit types This will prevent conflicts e.g. in code that deals with both h264 and hevc.	9 years ago
Anton Khirnov	251cbb4400	h264: create a new header for common h264 definitions Move the NAL unit types into it. This will allow to stop including the whole decoder-specific h264dec.h in some code that is unrelated to the decoder and only needs some enum values.	9 years ago
Anton Khirnov	9df889a5f1	h264: rename h264.[ch] to h264dec.[ch] This is more consistent with the naming of other decoders.	9 years ago
Anton Khirnov	debca90863	h264: store {curr,max}_pic_num in the per-slice context While the value of those variables will be constant for the whole frame, they are only used in two functions called from slice header decoding. Moving them to the per-slice context allows us to make the H264Context passed to slice_header_parse() constant.	9 years ago
Diego Biurrun	84b2d3fb68	h264: Drop unused function check_opcodes()	9 years ago
Anton Khirnov	bec993381c	h264: postpone generating the implicit MMCOs Do it right before the MMCOs are applied to the DPB. This will allow moving the frame_start() call out of the slice header parsing, since generating the implicit MMCOs needs to be done after frame_start().	9 years ago
Anton Khirnov	2d410ebbaa	h264: decode the MMCOs into per-slice contexts They are stored in the slice header, so technically they are per-slice (though they must be the same in every slice). This will simplify the following commits.	9 years ago
Anton Khirnov	39ab2ea531	h264: rename mmco_index to nb_mmco The variable stores the number of mmco entries, so the current name is misleading.	9 years ago
Anton Khirnov	ed9a20ebe4	h264: split reading the ref list modifications and actually building the ref list This will allow postponing the reference list construction (and by consequence some other functions, like frame_start) until the whole slice header has been parsed.	9 years ago
Anton Khirnov	7b50d60442	h264: call ff_h264_fill_mbaff_ref_list() when constructing the normal ref list There is no real reason to call it separately.	9 years ago
Anton Khirnov	b16e9b9ac9	h264: move initialising the explicit pred weight table for MBAFF Currently it's done in the code that initialises the ref list for MBAFF, which is not a logical place for it. Move it to the function that parses the pred table from the bitstream, which is analogous to what is done for the implicit weight table as well.	9 years ago
Anton Khirnov	b93c0aed79	h264: drop an outdated comment	9 years ago
Vittorio Giovara	41ed7ab45f	cosmetics: Fix spelling mistakes Signed-off-by: Diego Biurrun <diego@biurrun.de>	9 years ago
Anton Khirnov	89ae244e78	h264_refs: remove an unused parameter from ff_h264_fill_mbaff_ref_list()	9 years ago
Anton Khirnov	755f79f84c	h264_refs: make the H264Context const where possible	9 years ago
Anton Khirnov	a2fd547839	h264_refs: reorder functions to avoid forward declarations	9 years ago
Anton Khirnov	c8dcff0cdb	h264: factor out calculating the POC count into a separate file This will allow decoupling the parser from the decoder.	9 years ago
Anton Khirnov	3176217c60	h264: decouple h264_ps from the h264 decoder Make the SPS/PPS parsing independent of the H264Context, to allow decoupling the parser from the decoder. The change is modelled after the one done earlier for HEVC. Move the dequant buffers to the PPS to avoid complex checks whether they changed and an expensive copy for frame threads.	9 years ago
Anton Khirnov	e481458bc3	h264: factor out pred weight table parsing into a separate file This will allow decoupling the parser from the decoder.	9 years ago
Diego Biurrun	29c2d06d67	cosmetics: Drop empty comment lines	9 years ago
Diego Biurrun	2080bea4a7	h264_refs: Remove broken trace debug code	9 years ago
Anton Khirnov	9d74012761	h264: improve behaviour with invalid reference lists Before `741b494fa8`, when the reference list modification description was invalid, the code would substitute the corresponding reference from the initial ("default") reference list. After that commit, it will just return an error. Since there are apparently invalid samples in the wild that used to play fine with the old code, it is a good idea to re-add some sort of error resilience here. So, when the reference list modification results in a missing frame, substitute a previous reference frame for it. The relevant sample again decodes fine with the same output as previously.	9 years ago
Anton Khirnov	741b494fa8	h264: eliminate default_ref_list According to the spec, the reference list for a slice should be constructed by first generating an initial (what we now call "default") reference list and then optionally applying modifications to it. Our code has an optimization where the initial reference list is constructed for the first inter slice and then rebuilt for other slices if needed. This, however, adds complexity to the code, requires an extra 2.5kB array in the codec context and there is no reason to think that it has any positive effect on performance. Therefore, simplify the code by generating the reference list from scratch for each slice.	9 years ago
Anton Khirnov	6d4d3fee63	h264: make sure the current picture is not made a long ref multiple times Fixes possible invalid reads, once one of those refs is freed, but the others remain. CC: libav-stable@libav.org	10 years ago
Anton Khirnov	a0f2946068	h264: use properly allocated AVFrames	10 years ago
Vittorio Giovara	6a85dfc830	lavc: Replace av_dlog and tprintf with internal macros	10 years ago
Anton Khirnov	a12d3188cb	h264: use a smaller struct for the ref lists There is no need to store a whole H264Picture, with a full AVFrame embedded in it. This should allow getting rid of the embedded AVFrame later.	10 years ago
Anton Khirnov	f42485dbce	h264: use a separate GetBitContext for slice data	10 years ago
Anton Khirnov	95eb35f305	h264: move the ref lists variables into the per-slice context	10 years ago
Anton Khirnov	56febc993b	h264: move the slice type variables into the per-slice context	10 years ago
Anton Khirnov	92c6c2a605	h264: split weighted pred-related vars into per-slice context	10 years ago
Gabriel Dume	4b1f5e5090	cosmetics: Write NULL pointer inequality checks more compactly Signed-off-by: Diego Biurrun <diego@biurrun.de>	11 years ago
Gabriel Dume	f929ab0569	cosmetics: Write NULL pointer equality checks more compactly Signed-off-by: Diego Biurrun <diego@biurrun.de>	11 years ago
Diego Biurrun	cba4e6062a	More correct printf format specifiers This avoids compiler warnings about argument and specifier type mismatch.	11 years ago
Vittorio Giovara	e0c16e4e32	mpegvideo: move mpegvideo formats-related defines to mpegutils.h	11 years ago
Vittorio Giovara	9b749c8274	h264: move relevant fields from Picture to H264Picture	11 years ago
Vittorio Giovara	f4bd9fe326	h264: fix clang warning about uninitialized variable	11 years ago
Vittorio Giovara	17a63ff0cd	h264: update flag name in ff_h264_decode_ref_pic_list_reordering() This is the name used in the specifications.	11 years ago
Luca Barbato	f8c507f44b	h264: Refactor ff_h264_decode_ref_pic_list_reordering In preparation for MVC support.	11 years ago
Anton Khirnov	98cc9efc5f	h264: prevent two matching fields from being both a short/long ref combination Fixes possible access to freed memory. Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind CC:libav-stable@libav.org	11 years ago
Anton Khirnov	4d388c0cd0	h264_refs: make sure not to write over the bounds of the default ref list Fixes invalid writes. Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind CC:libav-stable@libav.org	11 years ago
Anton Khirnov	a553c6a347	lavc: use buf[0] instead of data[0] in checks whether a frame is allocated data[0] may be NULL for valid frames with hwaccel pixel formats.	11 years ago
Diego Biurrun	2a61592573	avcodec: Remove some commented-out debug cruft	12 years ago
Diego Biurrun	1b6f84a986	h264_refs: Do not print check_opcodes() return value The return value provides no useful information and removing the printing avoids the following warning: libavcodec/h264_refs.c:788:15: warning: 'i' may be used uninitialized in this function [-Wuninitialized]	12 years ago
Anton Khirnov	7fa00653a5	h264: add a parameter to the FIELD_PICTURE macro. This way it does not look like a constant.	12 years ago
Anton Khirnov	c2597c5a0a	h264_refs: cosmetics, reformat	12 years ago

1 2

73 Commits (a37e84be69310cd7de9540c8bc194cb0a6d158ed)