| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
|
|
| |
Setting colormetry in caps for RGB have no meaning, but worst it
confuses the converters downstream.
https://bugzilla.gnome.org/show_bug.cgi?id=759624
|
|
|
|
|
|
|
|
|
| |
Especially in push mode we would completely ignore the size of the data chunk
when not stop position is given for the seek. Instead make sure that the end
offset is at most the end of the data chunk if known.
Without this we would output anything after the data chunk, possibly causing
loud noises if the media file is followed by an INFO chunk or an ID3 tag.
|
|
|
|
|
|
| |
We use that to signal "infinity", taking the difference between that and some
other value is not going to give us any useful result for the end offsets of
segments.
|
|
|
|
|
|
|
|
| |
We check the stream profile and use the proper RTCP caps:
application/x-srtcp if we are using a secure profile and
application/x-rtcp otherwise.
https://bugzilla.gnome.org/show_bug.cgi?id=760556
|
|
|
|
|
| |
Post an element error when the CoreAudio device cannot be selected or opened.
Also ensure that we post a GST_ERROR with more detail.
|
|
|
|
|
|
|
|
| |
Even if we have more data queued up when flushing than the size of the data
chunk, don't process and output it. If the data size is known, this likely
contains another chunk (e.g. an INFO chunk) or things like ID3 tags. Just
outputting them as if they were data is going to cause unexpected behaviour
and unpleasant audio noises.
|
|
|
|
|
|
|
|
|
|
|
|
| |
SBC frame length calculation wasn't being rounded up to the nearest byte
(as specified in the A2DP 1.0 specification, section 12.9). This could
cause 'stereo' and 'joint stereo' mode SBC streams to have incorrectly
calculated frame lengths.
Incorrect frame length calculation causes frame coalescing to fail, as
subsequent frames in the stream aren't found in the expected locations.
https://bugzilla.gnome.org/show_bug.cgi?id=742446
|
|
|
|
|
|
| |
FALSE would mean FLOW_OK
https://bugzilla.gnome.org/show_bug.cgi?id=760666
|
|
|
|
|
|
|
| |
We call gst_rtp_buffer_get_payload() which creates a sub-buffer
of each input buffer, just to copy over metas, and then leak it.
https://bugzilla.gnome.org/show_bug.cgi?id=760289
|
|
|
|
|
| |
Previously we leaked them completely, but as they're specific to the caps
freeing them in stop() instead of finalize() makes most sense.
|
|
|
|
|
|
| |
multiple buffers
https://bugzilla.gnome.org/show_bug.cgi?id=758943
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
- set mute value at startup
- correct set and get mute functions
https://bugzilla.gnome.org/show_bug.cgi?id=755106
|
|
|
|
|
| |
When we are resurecting a buffer, don't block. instead let us copy a
buffer.
|
|
|
|
|
|
|
|
|
|
|
| |
there was an error
If GetStatus() fails, the status itself won't be very meaningful but we also
have to look at its return value. This fixes blocking pipelines when removing
sound devices or during other errors, where we wouldn't notice the error and
then wait forever.
https://bugzilla.gnome.org/show_bug.cgi?id=734098
|
|
|
|
|
|
|
|
|
| |
Actual code is checking for a NULL terminator and a ';' terminator,
for backward compat, in a chained way that cause all events being rejected.
The proper condition is to reject the events when terminator isn't
in ['\0', ';'] set.
https://bugzilla.gnome.org/show_bug.cgi?id=758151
|
|
|
|
|
| |
Add a simple test that checks proper serialization/deserialization
of custom events with rtpgstpay and rtpgstdepay.
|
|
|
|
|
|
|
| |
We would queue 5 consective packets before considering a reset and a proper
discont here. Instead of expecting the next output packet to have the current
seqnum (i.e. the fifth), expect it to have the first seqnum. Otherwise we're
going to drop all queued up packets.
|
|
|
|
|
|
|
|
|
|
|
| |
auds.blockalign is set once the first caps arrive. If
gst_avi_mux_stop_file() is called before this happens then auds.blockalign
is zero and gst_avi_mux_audsink_set_fields() cause a crash:
[...]
avipad->parent.hdr.rate = avipad->auds.av_bps / avipad->auds.blockalign;
[...]
https://bugzilla.gnome.org/show_bug.cgi?id=758912
|
|
|
|
|
|
|
| |
Makes sure upstream will unblock and return the thread so that
seeking can continue
https://bugzilla.gnome.org/show_bug.cgi?id=758861
|
|
|
|
|
|
|
|
|
|
|
|
| |
negative and positive numbers
This fixes seeking if the first entries in the samples table are negative. The
binary search would always fail on this as the array would not be sorted if
interpreting the negative numbers as huge positive numbers. This caused us to
always output buffers from the beginning after a seek instead of close to the
seek position.
Also add a case to the comparison function for equality.
|
|
|
|
|
|
| |
inactive_cond is not being cleared resulting in memory leak.
https://bugzilla.gnome.org/show_bug.cgi?id=757924
|
| |
|
|
|
|
|
|
|
| |
When the cenc aux info index is out of moof boundaries, keep track of
it and parse the beginning of the mdat box, before the first sample.
https://bugzilla.gnome.org/show_bug.cgi?id=755614
|
|
|
|
|
| |
This is needed to make sure that child processes don't inherit the video
device fd which can cause problems with some drivers.
|
|
|
|
| |
Don't write time as e.g. 11:9:42
|
|
|
|
|
|
| |
Or else flvdemux don't understand it
https://bugzilla.gnome.org/show_bug.cgi?id=754435
|
|
|
|
|
|
| |
Add a basic test using speex data to verify timestamping.
https://bugzilla.gnome.org/show_bug.cgi?id=754435
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For the MS/VfW codec ids, we want to write DTS timestamps instead
of PTS because that's what everyone else seems to do (and it's also
how it is in AVI). So for those input formats we use the buffer DTS
instead of the PTS. However, if there's no DTS set but only the PTS
then just take the PTS instead of dropping the input buffer. This
is useful especially for I-frame only codecs like JPEG and huffyuv,
but should also be fine as fallback in general.
Fixes regression with input JPEG frames that only have PTS set on them.
https://bugzilla.gnome.org/show_bug.cgi?id=756967
|
|
|
|
|
|
|
| |
Negotiation to audio/x-raw,format=S8 was not possible because S8 does
not have a bit order so we ended up doing `if (!entry.fourcc) goto refuse_caps;`
https://bugzilla.gnome.org/show_bug.cgi?id=756387
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=756388
|
|
|
|
|
|
| |
actually releases pads
https://bugzilla.gnome.org/show_bug.cgi?id=753622
|
|
|
|
|
|
|
|
|
|
| |
Instead, delay it until all request pads have been released. This is
because the release_pad() vfunc requires the multiqueue and muxer to
be there in order to release their request pads as well. If those
elements are destroyed earlier, release_pad() does not work, no
pads are released and some resources are leaked.
https://bugzilla.gnome.org/show_bug.cgi?id=753622
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We have to reverse all samples in a buffer before processing them to properly
have continuous data from one buffer to another. As a result we will have a
negative applied rate and a rate of 1.0.
Also make sure that input buffers are correctly clipped to the segment,
otherwise our calculations are going to go wrong.
Also copy over the segment event's sequence number to the output segment while
we're at it.
https://bugzilla.gnome.org/show_bug.cgi?id=757033
|
| |
|
|
|
|
|
|
|
| |
It was always being set to 0, making the resulting stream broken
for the receiver
https://bugzilla.gnome.org/show_bug.cgi?id=756422
|
|
|
|
|
|
| |
It's not available on older OSX and we can as well use memchr() here.
https://bugzilla.gnome.org/show_bug.cgi?id=756154
|
|
|
|
|
|
| |
mapped buffer is not being unmapped during failures
https://bugzilla.gnome.org/show_bug.cgi?id=756231
|
|
|
|
|
|
| |
Free the event after being handled to prevent memory leak.
https://bugzilla.gnome.org/show_bug.cgi?id=756799
|
|
|
|
|
|
|
|
|
|
|
| |
If the QtDemuxStream are re-used they may already have caps which used
to be leaked.
Reproduced using the
validate.dash.playback.seek_forward.dash_exMPD_BIP_TC1 validate
scenario.
https://bugzilla.gnome.org/show_bug.cgi?id=756561
|
|
|
|
|
|
| |
Free the stream and its sub items instead of just the stream
https://bugzilla.gnome.org/show_bug.cgi?id=756544
|
|
|
|
|
|
|
| |
When getting date from taglist, the memory should be freed after
using it.
https://bugzilla.gnome.org/show_bug.cgi?id=756171
|
|
|
|
|
|
|
| |
When getting sample from taglist, the memory should be freed after
using it.
https://bugzilla.gnome.org/show_bug.cgi?id=756068
|