| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
The contents of this field are not defined for decoding. Use
pkt_timebase, which is the timebase of demuxed packets.
Drop a tautological av_packet_rescale_ts() call, as the stream and
decoder timebases are the same.
|
|
|
|
| |
It is recommended for callers to set it, though not required.
|
| |
|
|
|
|
| |
Signed-off-by: James Almer <jamrial@gmail.com>
|
|
|
|
|
|
|
|
| |
Fixes: NULL pointer dereference
Fixes: 59359/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_AV1_fuzzer-6726080594313216
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: James Almer <jamrial@gmail.com>
|
| |
|
|
|
|
|
|
|
| |
All but one read are byte aligned, so there's no point in using the
GetBitContext API.
Signed-off-by: James Almer <jamrial@gmail.com>
|
| |
|
| |
|
|
|
|
| |
And make output frame durations always same.
|
| |
|
|
|
|
|
| |
* remove logging to NULL
* remove extraneous logging
|
|
|
|
| |
It makes no sense to do so.
|
|
|
|
|
|
| |
Current code marks the output stream as finished and waits for a flush
packet, but that is both unnecessary and suspect, as in theory nothing
should be sent to a finished stream - not even flush packets.
|
| |
|
|
|
|
| |
It is no longer accessed outside of ffmpeg_filter.
|
|
|
|
| |
It is no longer used outside ffmpeg_filter.
|
|
|
|
|
| |
Filtergraphs with no inputs are initialized as soon as all their outputs
are bound, so this code should not be reachable.
|
|
|
|
|
| |
There is no way for a filtergraph to have all input parameters, yet not
be configured, so this code should not be reachable.
|
|
|
|
|
|
| |
Make all relevant state per-filtergraph input, rather than per-input
stream. Refactor the code to make it work and avoid leaking memory when
a single subtitle stream is sent to multiple filters.
|
|
|
|
|
|
| |
Set them in ifilter_parameters_from_dec(), similarly to audio/video
streams. This reduces the extent to which sub2video filters need to be
treated specially.
|
|
|
|
|
|
| |
This function should not take an InputStream, as it only uses it to get
the InputFile and the timebase. Pass those directly instead and avoid
confusion over dealing with multiple InputStreams.
|
|
|
|
|
|
| |
This queue should be associated with a specific filtergraph input - if
a subtitle stream is sent to multiple filters then each should have its
own queue.
|
|
|
|
|
|
|
|
|
| |
This code is a sub2video analogue of ifilter_send_frame(), so it
properly belongs to the filtering code.
Note that using sub2video with more than one target for a given input
subtitle stream is currently broken and this commit does not change
that. It will be addressed in following commits.
|
| |
|
|
|
|
|
|
| |
When the filtergraph has no inputs, it can be configured immediately
when all its outputs are bound to output streams. This will simplify
treating some corner cases.
|
|
|
|
| |
That is a more appropriate place for it.
|
|
|
|
|
| |
While the new function is trivial for now, it will become more useful in
future commits.
|
|
|
|
|
|
| |
This way the list of filtergraph inputs/outputs is always known after
FilterGraph creation. This will allow treating simple and complex
filtergraphs in a more uniform manner.
|
|
|
|
| |
Needed by the following commit.
|
|
|
|
|
|
| |
InputStream
Will be useful in future commits.
|
|
|
|
| |
Not the entire AVFilterInOut. This is simpler.
|
|
|
|
|
|
| |
Currently NULL would be passed for simple filtergraphs, which would
make the filter code extract the graph description from the output
stream when needed. This is unnecessarily convoluted.
|
|
|
|
|
| |
Do not use an unrelated graph property, which will change in future
commits.
|
| |
|
|
|
|
|
| |
This way ost_add() knows about the complex filtergraph it is fed from,
which will become useful in future commits.
|
|
|
|
|
|
|
|
|
|
|
|
| |
The existing DecklinkQueue implementation was using the PacketList
structure but wasn't using the standard avpriv_packet_list_get and
avpriv_packet_list_put functions. Convert to using them so we
eliminate the duplicate logic, per Marton Balint's suggestion.
Updated to reflect feedback from Marton Balint provided on 05/11/23.
Signed-off-by: Devin Heitmueller <dheitmueller@ltnglobal.com>
Signed-off-by: Marton Balint <cus@passwd.hu>
|
|
|
|
|
|
|
|
|
|
| |
Check init_get_bits' result for NULL, to avoid dereferencing a NULL
pointer later (CWE-476).
Without this, a segfault happens when trying to decode a handcrafted
ogg-flac file with an absurdly long (e.g. 268435455 bytes) ogg header.
Co-authored-by: James Almer <jamrial@gmail.com>
Signed-off-by: Paul Arzelier <paul.arzelier@free.fr>
|
| |
|
| |
|
|
|
|
| |
This helps compiler figure out it can unroll loop and give 4x speedup.
|
| |
|
|
|
|
|
|
| |
Optimize the put and avg filtering for 8x8 chroma blocks
Signed-off-by: Arnie Chang <arnie.chang@sifive.com>
|
| |
|
|
|
|
| |
Signed-off-by: Ronald S. Bultje <rsbultje@gmail.com>
|
|
|
|
|
| |
Also produce output video timestamps that are consistent with
10 frames per second.
|
| |
|
| |
|
| |
|
| |
|