- 28 Aug, 2019 1 commit
-
-
Paul B Mahol authored
Fixes timecode calculation for NTSC frame rates. Fixes ticket #5978.
-
- 25 Aug, 2019 1 commit
-
-
James Almer authored
Signed-off-by:
James Almer <jamrial@gmail.com>
-
- 22 Jul, 2019 1 commit
-
-
Gyan Doshi authored
A value of zero for sample_count in trun box is not prohibited by 14496-12 section 8.8.8. 4a9d32ba disallowed this which led the demuxer to error out when reading the header of valid files.
-
- 25 Jun, 2019 1 commit
-
-
Michael Niedermayer authored
Fixes: Assertion failure Fixes: crbug971646.mp4 Reported-by:
Matt Wolenetz <wolenetz@google.com> Reviewed-by:
Matt Wolenetz <wolenetz@google.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 03 May, 2019 1 commit
-
-
Gyan Doshi authored
Ever since 59ad5046, the demuxer expects its seek arg to be PTS and adjusts internally to search index by DTS.
-
- 24 Apr, 2019 1 commit
-
-
Michael Niedermayer authored
Fixes: Assertion failure Fixes: clusterfuzz-testcase-minimized-media_pipeline_integration_fuzzer-5683096400822272 Found-by: Clusterfuzz Reported-by:
Dan Sanders <sandersd@google.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 23 Apr, 2019 1 commit
-
-
Dan Sanders authored
It's this or add overflow detection in mov_read_header(). Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 25 Mar, 2019 1 commit
-
-
Michael Niedermayer authored
No testcase Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 21 Mar, 2019 1 commit
-
-
Carl Eugen Hoyos authored
Reviewed-by: Lauri Kasanen Reviewed-by: Tomas Härdin
-
- 26 Feb, 2019 1 commit
-
-
Justin Ruggles authored
Empty edits can occur at any position within the edit list except for at the end. Empty edits in the middle should not impact the reported stream start_time or the video PTS adjustment, so only include empty edits at the start of the list in empty_edits_sum_duration. Signed-off-by:
Vittorio Giovara <vittorio.giovara@gmail.com>
-
- 14 Feb, 2019 1 commit
-
-
Michael Niedermayer authored
Fixes: NULL pointer dereference Fixes: clusterfuzz-testcase-minimized-audio_decoder_fuzzer-5634316373721088 Reported-by:
Chris Cunningham <chcunningham@google.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 11 Feb, 2019 2 commits
-
-
Charles Liu authored
Binary searching would hang if the fragment items do NOT have timestamp for the specified stream. For example, a fmp4 consists of separated 'moof' boxes for each track, and separated 'sidx' for each segment, but no 'mfra' box. Then every fragment item only have the timestamp for one of its tracks. Example: ffmpeg -f lavfi -i testsrc -f lavfi -i sine -movflags dash+frag_keyframe+skip_trailer+separate_moof -t 1 out.mp4 ffmpeg -ss 0.5 -i out.mp4 -f null none Also fixes the hang in ticket #7572, but not the reason for having AV_NOPTS_VALUE timestamps there. Signed-off-by:
Charles Liu <liuchh83@gmail.com> Signed-off-by:
Marton Balint <cus@passwd.hu>
-
James Almer authored
Simplifies code. Reviewed-by:
Michael Niedermayer <michaelni@gmx.at> Signed-off-by:
James Almer <jamrial@gmail.com>
-
- 08 Feb, 2019 2 commits
-
-
chcunningham authored
Bad content may contain stsc boxes with a first_chunk index that exceeds stco.entries (chunk_count). This ammends the existing check to include cases where chunk_count == 0. It also patches up the case when stsc refers to unknown chunks, but stts has no samples (so we can simply ignore stsc). Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
chcunningham authored
Detecting missing tfhd avoids re-using tfhd track info from the previous moof. For files with multiple tracks, this may make a mess of the avindex and fragindex, which can later trigger av_assert0 in mov_read_trun(). Reviewed-by:
Derek Buitenhuis <derek.buitenhuis@gmail.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 14 Jan, 2019 1 commit
-
-
Carl Eugen Hoyos authored
This is what several other players do and what FFmpeg already does for the sidx atom. Fixes ticket #7679.
-
- 02 Jan, 2019 1 commit
-
-
Derek Buitenhuis authored
This breaks totally valid files that get caught in its heuristic. This, according to the commit message, is my own doing, having asked Michael to implement this check and providing a sample that was "wrong". I am now atoning for my sins, and removing this hack, having seen the light (aka that this was silly to do in the first place). Resotores correct behavior on valid files. This reverts commit 8e5e84c2. Signed-off-by:
Derek Buitenhuis <derek.buitenhuis@gmail.com>
-
- 20 Dec, 2018 1 commit
-
-
Jacob Trimble authored
When reading frames, we need to use the fragment for the correct stream. Sometimes the "current" fragment is not the same as the one the frame is for. Found by Chromium's ClusterFuzz: https://crbug.com/906392 and https://crbug.com/915524Signed-off-by:
Jacob Trimble <modmaker@google.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 19 Dec, 2018 1 commit
-
-
Michael Niedermayer authored
Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 17 Dec, 2018 1 commit
-
-
Jan Ekström authored
-
- 16 Dec, 2018 1 commit
-
-
chcunningham authored
Chromium fuzzing produced a whacky file with extra tkhds. This caused an AVStream that was already in use to be corrupted by assigning it a new id, which blows up later in mov_read_trun because the MOVFragmentStreamInfo.index_entry now points OOB. Reviewed-by:
Baptiste Coudurier <baptiste.coudurier@gmail.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 09 Sep, 2018 1 commit
-
-
Justin Ruggles authored
ISOBMFF does not allow AudioSampleEntryV1 in stsd version 0, so assume the descriptor format is QTFF SoundDescriptionV1. ISOBMFF does not define a version 2. This fixes audio decoding for some MP4 files generated with Apple tools. The additional fields present in SoundDescriptionV1/V2 need to be read in order to correctly read additional boxes that contain information required for decoding the stream. Fixes #7376. Also see: https://github.com/HandBrake/HandBrake/issues/1555Signed-off-by:
Derek Buitenhuis <derek.buitenhuis@gmail.com>
-
- 08 Sep, 2018 2 commits
-
-
John Stebbins authored
Since it is performed in all cases now. Signed-off-by:
James Almer <jamrial@gmail.com>
-
John Stebbins authored
This allows for validation of the track type Signed-off-by:
James Almer <jamrial@gmail.com>
-
- 01 Sep, 2018 1 commit
-
-
Dale Curtis authored
Entries are always at least 8 bytes per the parsing code, so if we see an impossible entry count avoid massive allocations. This is similar to an existing check in mov_read_stsc(). Since ff_mov_read_stsd_entries() does eof checks, an alternative approach could be to clamp the entry count to atom.size / 8. Signed-off-by:
Dale Curtis <dalecurtis@chromium.org> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 23 Aug, 2018 1 commit
-
-
Dale Curtis authored
This field is a uint16_t, see docs: http://opus-codec.org/docs/opus_in_isobmff.html#4.3.2Signed-off-by:
Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
James Almer <jamrial@gmail.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 22 Aug, 2018 1 commit
-
-
Jacob Trimble authored
If there is a saio/saiz in clear content, we shouldn't create the encryption index if we don't already have one. Otherwise it will confuse the cenc_filter. The changed method is also used for senc atoms, but they should not appear in clear content. Found by Chromium's ClusterFuzz: https://crbug.com/873432Signed-off-by:
Jacob Trimble <modmaker@google.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 17 Aug, 2018 1 commit
-
-
James Almer authored
This will get ISOBMFF and Matroska up to date with the revised AV1 Codec Configuration Box spec. For now keep propagating raw OBUs as extradata until all libavcodec modules are adapted to handle AV1CodecConfigurationRecord formatted extradata. Tested-by:
Thomas Daede <bztdlinux@gmail.com> Signed-off-by:
James Almer <jamrial@gmail.com>
-
- 11 Aug, 2018 1 commit
-
-
Sasi Inguva authored
Fixes vorbis mp4 audio files, with edit list specified. Since st->skip_samples is not set in case of vorbis , ffmpeg computes the start_time as negative. Signed-off-by:
Sasi Inguva <isasi@google.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 08 Aug, 2018 1 commit
-
-
Carl Eugen Hoyos authored
The mp4 registration authority accepted a code-point for Dolby Vision HEVC that was already used as a fourcc for DVCPRO 100. Fixes ticket #7347.
-
- 26 Jul, 2018 1 commit
-
-
Michael Niedermayer authored
Fixes: 2018-0721-sample Fixes: null pointer dereference Found-by: Nikita Knyzhov (knnikita@yandex.ru) Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 20 Jul, 2018 1 commit
-
-
James Almer authored
Signed-off-by:
James Almer <jamrial@gmail.com>
-
- 16 Jul, 2018 1 commit
-
-
Gyan Doshi authored
6 FATE references updated. Fixes #7104
-
- 13 Jul, 2018 2 commits
-
-
Michael Niedermayer authored
Reviewed-by:
Derek Buitenhuis <derek.buitenhuis@gmail.com> Reviewed-by:
Sasi Inguva <isasi@google.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
Michael Niedermayer authored
0.266 <- 0.299 sec (this is time ffmpeg so containing alot other things) Sample for benchmark was: ffmpeg -f rawvideo -pix_fmt yuv420p -s 32x32 -i /dev/zero -t 24:00:00.00 out.mp4 Reviewed-by:
Derek Buitenhuis <derek.buitenhuis@gmail.com> Reviewed-by:
Sasi Inguva <isasi@google.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 11 Jul, 2018 2 commits
-
-
Michael Niedermayer authored
Reviewed-by:
Derek Buitenhuis <derek.buitenhuis@gmail.com> Reviewed-by:
Sasi Inguva <isasi@google.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
Michael Niedermayer authored
0.324 <-0.491 sec Reviewed-by:
Derek Buitenhuis <derek.buitenhuis@gmail.com> Reviewed-by:
Sasi Inguva <isasi@google.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 03 Jul, 2018 1 commit
-
-
Jacob Trimble authored
This exposes encryption info from the container to the app. This includes key ID, IV, and subsample byte ranges. The info is passed using the new side-data AV_PKT_DATA_ENCRYPTION_DATA and AV_PKT_DATA_ENCRYPTION_INIT_DATA. Signed-off-by:
Jacob Trimble <modmaker@google.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-
- 19 Jun, 2018 1 commit
-
-
Carl Eugen Hoyos authored
Fixes ticket #6433.
-
- 14 Jun, 2018 1 commit
-
-
Michael Niedermayer authored
Reviewed-by:
Sasi Inguva <isasi@google.com> Signed-off-by:
Michael Niedermayer <michael@niedermayer.cc>
-