| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
Introduced by c4c9fe60b pcapparse: Take buffer directly from the adapter
Using gst_adapter_take_buffer_fast() can lead to buffers that are
made up of multiple memories with the first memory smaller than the
RTP header size, which violates assumptions GstRtpBaseDepayloader
makes, namely that the complete RTP header will be in the first
memory. This leads to such packets being dropped when feeding
them from pcapparse to RTP depayloaders. Use take_buffer() so
we get buffers with a single memory.
|
|
|
|
|
|
|
|
|
|
| |
Introduced by c4c9fe60b pcapparse: Take buffer directly from the adapter
Flush any trailing bytes after the payload from the adapter as well,
otherwise we'll read a bogus packet size from the adapter next and
then everything goes downhill from there.
https://bugzilla.gnome.org/show_bug.cgi?id=751879
|
|
|
|
|
|
|
| |
gst_ps_demux_get_stream() could return NULL when it is unknown stream,
check this hasn't happened before dereferencing the returned pointer.
https://bugzilla.gnome.org/show_bug.cgi?id=751528
|
|
|
|
|
|
|
|
|
| |
Can be used to fix misbehaving sinks. It will pass through all buffers
until it encounters GST_FLOW_ERROR or GST_FLOW_NOT_NEGOTIATED (configurable).
At that point it will unref the buffers and return GST_FLOW_NOT_LINKED
(configurable) - until the next READY_TO_PAUSED or FLUSH_STOP.
https://bugzilla.gnome.org/show_bug.cgi?id=750098
|
|
|
|
| |
Update ORC files with newest ORC to fix backup code generation error.
|
|
|
|
|
|
|
|
| |
This should hopefully allow even the most challenged static
code analyzer to figure out that it's all fine. Also makes
the flow clearer.
https://bugzilla.gnome.org/show_bug.cgi?id=751305
|
|
|
|
|
|
| |
trivial patch to add proper ( while checking for if(G_UNLIKELY())
https://bugzilla.gnome.org/show_bug.cgi?id=751305
|
| |
|
| |
|
|
|
|
|
|
|
| |
Move the pixel-aspect-ratio calculations higher up in caps
determination, so the results are available for a call to
gst_video_multiview_guess_half_aspect() when stereoscopic video
is detected.
|
|
|
|
|
|
| |
Use QOS messages to update rendered and dropped frame stats. This is
the only accurate method. The old method didn't take max-lateness and
latency into account.
|
|
|
|
|
|
|
| |
After few iteration, this variable became the same as dts. It's not
the min as the name says, but the dts of the current buffer. Simply
remove and place with dts. Also move the debug trace to actually
print the signed version of the running-time dts.
|
|
|
|
|
|
|
|
| |
after e000a6f0a4984fbe81190df6cd439c70dafde3d6, there is build error in bad plugins
this happens because, GST_CLOCK_STIME_IS_VALID () is being checked for pad_data
but it expects a GstClockTime parameter. Changing the check to 'dts'
https://bugzilla.gnome.org/show_bug.cgi?id=750961
|
|
|
|
| |
It is not bound between 0 and 10000 anymore.
|
|
|
|
|
|
|
|
| |
The segment should start at first PTS, and the vairable name lower_pts
state so correctly. Though we where using the first DTS instead. This
could lead to small desynchronization of video stream.
https://bugzilla.gnome.org/show_bug.cgi?id=740575
|
|
|
|
|
|
|
|
|
|
| |
Use the saved DTS, make it signed and pass that to the stream muxer. This
preserves the running time sign. All usage of -1 as invalid TS are now
replaced with G_MININT64. Negative values will be seen as wrap-around
point, but the delta between PTS and DTS will remain correct. Demuxers
don't care about absolute values, they only cares about deltas.
https://bugzilla.gnome.org/show_bug.cgi?id=740575
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=740575
|
|
|
|
|
|
|
|
| |
There was code to detect backward dts, but the marker min_dts
was never set. Setting it enable this feature that prevents
potential integer overflow when generating TS.
https://bugzilla.gnome.org/show_bug.cgi?id=740575
|
|
|
|
| |
Ran gst-indent on debugutilsbad.c, moved headers to noinst in Makefile.am
|
|
|
|
| |
Pass any multiview info from the container or SEI info downstream
|
|
|
|
|
|
|
| |
Wait until at least one keyframe has been parsed before
deciding to switch to passthrough mode, in case the
stream contains SEI messages that supplement the output
caps - for example by providing stereoscopic information
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=746765
|
|
|
|
|
|
|
|
|
|
|
|
| |
We were off by one byte in the matching
It should be (using 24 bit matching):
* startcode : 0000 0000 0000 0000 1000 00xx
* mask (bin) : 1111 1111 1111 1111 1111 1100
* mask (hex) : f f f f f c
* match : 0 0 0 0 8 0
https://bugzilla.gnome.org/show_bug.cgi?id=750685
|
|
|
|
|
|
| |
Rebased for latest master by Nicolas Huet <nicolas.huet@parrot.com>.
https://bugzilla.gnome.org/show_bug.cgi?id=745748
|
| |
|
|
|
|
| |
The naive port to 1.0 from years ago was silly, make it better.
|
|
|
|
|
|
|
| |
Rename template to caps to keep the original intention of the code after
commit b4c9aa1c
CID #1304674
|
|
|
|
| |
This reverts commit 0635acfec041b1c664bc0770839b1a576e3598b1.
|
|
|
|
|
|
|
| |
After commit b4c9aa1c308f88bf4e1f69ab0156ed9f99815e8e template will always be
NULL. The if conditional will always be FALSE, so removing it.
CID #1304674
|
| |
|
|
|
|
| |
Function goes to done before the value set in start_offset is ever used.
|
|
|
|
|
|
|
|
|
|
| |
In case of the videomark being partially or fully outside,
an error was bein thrown saying, mark width is more than video width.
And when the width, offset properties are set to maximum it resulted in crash.
Instead of throwing error, added logic to detect the mark
in case of partial visibility or dont show the mark when it is outside.
https://bugzilla.gnome.org/show_bug.cgi?id=743908
|
|
|
|
|
|
| |
When the pattern offset is outside the video, the print error message
https://bugzilla.gnome.org/show_bug.cgi?id=743908
|
|
|
|
| |
Avoid repeating the same pad creation code everywhere
|
|
|
|
| |
In all these cases ret is set but overwritten before the return of the function
|
|
|
|
| |
Data might not exist anymore
|
|
|
|
|
|
|
|
|
|
| |
In case of the videomark being partially or fully outside, an error was being
thrown saying the mark width is more than video width. And when the width,
offset properties are set to maximum it resulted in crash. Instead of throwing
an error, add logic to detect the mark in case of partial visibility or don't
show the mark when it is outside.
https://bugzilla.gnome.org/show_bug.cgi?id=743908
|
|
|
|
|
|
|
|
|
| |
Chinese broadcaster encapsulate AVS video codec into MPEG2-TS. They
use the stream_id 0x42 to identify AVS video streams. It should be noted
that this id is currently within the ISO reserved range, hence it's
utilisation is unofficial.
https://bugzilla.gnome.org/show_bug.cgi?id=727731
|
|
|
|
|
|
| |
Don't force it anywhere
https://bugzilla.gnome.org/show_bug.cgi?id=750252
|
|
|
|
| |
This is the same number that was used to allocate the buffer
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=742875
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=742875
|
|
|
|
|
|
|
|
| |
Value of res is reset to FALSE in each iteration of the while loop. We want to
conserve TRUE if any pad event succeeded until we arrive to done.
Also, buf is set to the value of *outbuf twice. Removing the first assignment
since the second one is outside of a conditional.
|
|
|
|
|
| |
buf is incremented just before returning, this new value is never used.
Removing unused assignment.
|
|
|
|
|
|
|
| |
When factor property is set to 0, transform just returns.
Adjust the minimum value to 1.
https://bugzilla.gnome.org/show_bug.cgi?id=743907
|
|
|
|
| |
Otherwise we divide by zero.
|
| |
|
|
|
|
|
|
| |
We were using the wrong formula
https://bugzilla.gnome.org/show_bug.cgi?id=749634
|
|
|
|
|
|
|
|
| |
Like SPS/PPS they do contain information which will be needed to
decode the following data (as per definition of the flag)
Also ensures that the series of SPS/PPS/SEI NALU before a keyframe
can be considered as one contiguous header
|
|
|
|
|
|
|
| |
In the same way we do it for the DELTA_UNIT flag
This allows downstream elements to know whether a given mpeg-ts
packet contains a corresponding HEADER elementary unit
|