| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Update old references for decodebin2 to decodebin.
|
|
|
|
|
|
|
|
|
| |
Free the existing descriptor array, if any, before replacing it.
Fix leaks with the
validate.file.playback.scrub_forward_seeking.test-mpeg2-mp3_mxf scenario.
https://bugzilla.gnome.org/show_bug.cgi?id=748580
|
|
|
|
| |
This is not needed any longer.
|
|
|
|
|
|
|
|
|
|
| |
Reset the internal segment before freeing it.
mxf_index_table_segment_parse() allocates data inside the segment
(like segment->delta_entries) which have to be freed using
mxf_index_table_segment_reset().
https://bugzilla.gnome.org/show_bug.cgi?id=746803
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=744572
|
| |
|
|
|
|
| |
No need to store the GstFlowReturn when we can check if it is not OK directly
|
|
|
|
|
| |
Having a variable named ret in a static void function is very confusing since
this usually is to store what the function will return.
|
|
|
|
|
|
|
| |
Value stored in ret will be ovewritten in the next iteration of the loop. Which
means it is never used.
Plus a style issue to make gst-indent happy and allow the commit.
|
|
|
|
|
| |
This speeds up seeking drastically.
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=732294
|
|
|
|
|
|
|
|
| |
There are unnecessary definitions for disabling deprecation warnings.
Since GLIB_DISABLE_DEPRECATION_WARNINGS is not needed anymore in these files,
removing the same.
https://bugzilla.gnome.org/show_bug.cgi?id=737559
|
|
|
|
| |
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=733123
|
|
|
|
| |
it isn't necessary anymore and is already unused
|
| |
|
|
|
|
|
|
| |
Removes flow return combination code to use the newly added GstFlowCombiner
https://bugzilla.gnome.org/show_bug.cgi?id=709224
|
| |
|
|
|
|
|
|
|
|
|
| |
Testing mpegversion when mpegaudioversion was likely meant.
Similar tests in sys/androidmedia/gstamcaudiodec.c also test
mpegaudioversion with the same conditional code.
Coverity 206071
|
|
|
|
|
|
| |
Use a placeholder value in that case, it's better than crashing.
Coverity 1139697
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This component is dereferenced, and later code checking for
NULL in particular cases implies it can be NULL. This likely
does not fix the coverity warning as it was seeing another
path setting component to NULL explicitely, but this was
spotted by looking at:
Coverity 1139736
Which is actually OK from what I can see since the actual
dereference of the explicit NULL pointer will not happen
if the condition that led to the NULL pointer assignment
is met, since the assignment and defeference have mutually
exclusive tests.
|
|
|
|
|
|
|
|
| |
We want to check whether the rate is different. We check changes in
numerator *and* denominator.
CID #1139631
CID #1139642
|
|
|
|
|
|
| |
And properly cleanup/reset the segment before returning on errors
CID #206012
|
|
|
|
|
|
|
| |
a guint will always smaller or equal to the maximum value it can
contain
CID #206049
|
|
|
|
|
|
|
|
|
| |
Also unref buffers on error, as it seems to be done in one, but
not all, error paths.
The NULL pointer part is Coverity 206112
https://bugzilla.gnome.org/show_bug.cgi?id=727889
|
|
|
|
| |
Just to be on the safe side.
|
|
|
|
|
| |
If a KEY_UNIT seek was requested, adjust segment
start to position of the key frame.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixes multiple seeking issues. When doing ACCURATE or normal
non-KEYUNIT seeks, mxfdemux would just send data from the
edit unit that covered the seek position, whether that's
a keyframe or not. Decoders would only output things from
the next keyframe then, which means there's a gap between
the start of the segment and the first decoded data in
some cases. In combination with gst-editing-services this
might result in a frozen picture for the duration of that
gap at the beginning (if videorate fixes up the first
buffer's start timestamp to cover the entire gap), or
a black frame (if no videorate is used and videomixer
fills the gap). Also fixes A/V sync issue when requesting
a KEYUNIT seek.
|
|
|
|
| |
Coverity 1139645
|
|
|
|
|
|
|
| |
When the scheduling query results has GST_SCHEDULING_FLAG_SEQUENTIAL set
in its flags don't go into pull mode to prevent over-eager seeking.
https://bugzilla.gnome.org/show_bug.cgi?id=722934
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Don't leak demuxer tag list.
|
|
|
|
|
|
|
|
|
|
| |
In the sink event handler we end up sending multiple EOS
events per pad. Don't return FALSE when sending the
second EOS on an already-EOS pad fails. Not sure if there
was a reason for sending a second EOS, so leaving the
code in there for now, but assume all went fine if there
are source pads, which is slightly less wrong than before.
This function needs work.
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=693322
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
Bad things will happen otherwise, like reading a complete,
multiple GB large video track into a single buffer.
Conflicts:
gst/mxf/mxfdemux.c
|
| |
|
|
|
|
|
| |
This does not add support for actually producing filler data
if there's a filler component in a timeline.
|
| |
|
|
|
|
|
| |
Conflicts:
gst/mxf/mxfmpeg.c
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=687520
|
|
|
|
|
|
|
|
|
| |
Also ports mxfmux to 1.0 to the extent that it compiles, but is 100% untested,
so remains disabled.
Conflicts:
gst/mxf/mxfdemux.c
gst/mxf/mxfmux.c
|
|
|
|
|
| |
where possible. Avoids some string copies. Also re-indent
some stuff. Also some indent fixes here and there.
|
| |
|