mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2024-06-26 01:40:38 +00:00
Code Issues 1 Projects Releases Wiki Activity
115419 commits 44 branches 1505 tags 371 MiB
Commit graph

115419 commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Jan Alexander Steffens (heftig) dcada94046 rtmp2/client: Make sure 'reason' is not NULL 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1403>
 2022-11-07 12:31:07 +00:00
Jan Alexander Steffens (heftig) a13f234be3 rtmp2/client: Make sure 'desc' is not NULL 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1403>
 2022-11-07 12:31:07 +00:00
Jan Alexander Steffens (heftig) 2037f1ed0a rtmp2/client: Make sure 'code' is not NULL 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1403>
 2022-11-07 12:31:07 +00:00
Edward Hervey a100f36b69 webrtcbin: Don't duplicate enum string values 
Some were leaked when debugging was enabled. Instead just directly use the
static strings as-is.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3347>
 2022-11-07 11:21:00 +00:00
Edward Hervey 605cb6a4d4 gstpad: Avoid race in (un)setting EOS flag on sinkpads 
The scenario is the following:

* Thread 1 is pushing an EOS event on a sinkpad
* Thread 2 is pushing a STREAM_START event on the same sinkpad before Thread 1
returns. Note : It starts pushing the event after Thread 1 took the object lock.

There is a potential race between:

* The moment Thread 1 sets the EOS flag once it has finished sending the
event (via store_sticky_event). When it does that it has both the STREAM and
OBJECT lock

* The moment Thread 2 sends the STREAM_START event (Which should release that
EOS status), but removing the EOS flag is only done while holding the OBJECT
lock and not the STREAM_LOCK, which means it could be re-set by Thread 1 before
it then checks again the EOS flag (without the STREAM lock taken).

The EOS flag unsetting by STREAM_START should be done with the STREAM lock
taken, otherwise it will be racy.

Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1452

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3320>
 2022-11-07 06:28:39 +01:00
Nicolas Dufresne e81e212610 videocodectestsink: Add YUV422 support 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3331>
 2022-11-07 00:40:25 +00:00
Tim-Philipp Müller 62f8d292a1 gst-inspect: print doc urls for Rust plugins 
We have documentation for them now after all.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3346>
 2022-11-06 18:10:44 +00:00
Justin Chadwell fd96fc23c5 qtdemux: use unsigned int types to store result of QT_UINT32 
In a few cases throughout qtdemux, the results of QT_UINT32 were being
stored in a signed integer, which could cause subtle bugs in the case of
an integer overflow, even allowing the the result to equal a negative
number!

This patch prevents this by simply storing the results of this function
call properly in an unsigned integer type. Additionally, we fix up the
length checking with stsd parsing to prevent cases of child atoms
exceeding their parent atom sizes.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3344>
 2022-11-06 12:00:31 +00:00
Seungha Yang 0405e0cfc7 d3d11videosink: Always clear back buffer on resize 
Swapchain may not need to be resized if the size of backbuffer
is equal to the previous size. Then previously rendered frame will be stay
on the screen. Do clear back buffer whenever resize() is called

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3341>
 2022-11-05 17:23:24 +00:00
Edward Hervey 24a0e5bdfd subparse: Fix non-closed tag handling. 
Unclear what the goal was, but we could end up reading way past the next_tag.

Instead just move everything from after the end tag ('>') to the next_tag.

Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=53040

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3340>
 2022-11-05 16:19:53 +00:00
Nicolas Dufresne e283c99358 avutils: Add missing space in doc 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3330>
 2022-11-05 15:12:08 +00:00
Nicolas Dufresne 7b70de580c avviddec: Initialize std_compliance to default 
This was missed in the initial patch.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3330>
 2022-11-05 15:12:08 +00:00
Seungha Yang e1e9ee5506 qsv: Fix encoding error when input memory belongs to other GPU 
Copy frame if VA display of input buffer is different from that
of encoder, like we've been doing on Windows

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3335>
 2022-11-05 13:59:45 +00:00
Tim-Philipp Müller 04581fad6a identity: fix "handoff" signal docs 
The docs list an extra pad argument, which doesn't
match the actual signal function signature. Probably
a copy'n'paste mistake when copying things from fakesink.

Fixes #1546

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3337>
 2022-11-05 11:58:34 +00:00
He Junyan 1881d1826a va: baseenc: Do not import the VA surface from other display. 
Fixes: https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1167
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3322>
 2022-11-05 13:39:34 +08:00
He Junyan d51b29b1dc va: compositor: Do not use allocator and pool from other display. 
Just like the va decoder, the compositor should not use allocator
and pool from other display. Also, when importing the input buffer,
if it is from other display, we should fallback to memory copy.

Fixes: https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1167
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3322>
 2022-11-05 13:39:34 +08:00
He Junyan b9b211c1d1 va: basetransform: Do not use allocator and pool from other display. 
Just like the va decoder, the vpp and deinterlace should not use allocator
and pool from other display. Also, when importing the input buffer, if it
is from other display, we should fallback to memory copy.

Fixes: https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1167
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3322>
 2022-11-05 13:39:34 +08:00
He Junyan a8330686cb va: basedec: Do not use allocator and pool from other display. 
The command line such as:
  gst-launch-1.0 -vf filesrc location=1.264 ! h264parse !
  vah264dec ! varenderD129postproc ! fakesink
The decoder here gets the allocation proposal from the vpp which is
on another GPU device. The allocator and pool belong to other display
and should not be used in the decoder.

Fixes: https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1167
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3322>
 2022-11-05 13:39:34 +08:00
He Junyan 3875c420f7 libs: va: Add API to peek the va buffer's display. 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3322>
 2022-11-05 13:39:34 +08:00
He Junyan 464650a5d0 libs: va: Add API to peek the va memory's display. 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3322>
 2022-11-05 13:39:34 +08:00
He Junyan b1a98433f8 libs: va: Add API to peek the allocator's display. 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3322>
 2022-11-05 13:39:34 +08:00
He Junyan 028920cfc4 libs: va: Fix code style in gstvaallocator.h 
Replace all tabs with spaces.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3322>
 2022-11-05 13:39:34 +08:00
Seungha Yang 4db4f08227 d3dvideosink: Update plugin description 
Direct3D is too generic (there are various version numbers).
Clarify that this element is using the Direct3D9 API

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3333>
 2022-11-04 22:48:41 +00:00
Tim-Philipp Müller edc2e8312e qt: initialize GError properly in gst_qt_get_gl_wrapcontext() 
Spotted by Claus Stovgaard.

Fixes #1545

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3327>
 2022-11-04 20:39:16 +00:00
Stéphane Cerveau dd35ec3ddd gst-env: import exit from sys 
On windows the Cmd Prompt for VS 2019 complains
that exit is not defined.

  File "C:/data/gstreamer/gst-env.py", line 622, in <module>
    exit(subprocess.call(args, close_fds=False, env=env))

NameError: name 'exit' is not defined
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3324>
 2022-11-04 20:07:30 +00:00
Sebastian Dröge b368a5fcd2 qtmux: Add durations to raw audio buffers from the raw audio adapter in prefill mode 
This ensures that a duration can also be calculated and stored for the
last buffer at EOS.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3321>
 2022-11-04 19:02:22 +00:00
Sebastian Dröge 7b60e48c8c qtmux: Release object lock before posting an error message 
GST_ELEMENT_ERROR() also takes the object lock and this would then
deadlock.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3321>
 2022-11-04 19:02:22 +00:00
Edward Hervey 73762156f3 gst-validate-1.0: Don't leak arguments and context 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3319>
 2022-11-04 17:59:22 +00:00
Edward Hervey 3beec9bad3 validate-scenario: Don't leak temporary path value 
It's only use in the structure iterator

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3319>
 2022-11-04 17:59:22 +00:00
Edward Hervey 2114490b9b validate-scenario: Don't leak temporary local variables structures 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3319>
 2022-11-04 17:59:21 +00:00
Edward Hervey f4d0537b3e lv2: Don't leak plugin information on registration 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3319>
 2022-11-04 17:59:21 +00:00
Edward Hervey 685a4aaaa7 ladspa: Don't leak plugin information on registration 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3319>
 2022-11-04 17:59:21 +00:00
Edward Hervey 8ca2a2a230 fdkaacenc: Properly terminate GEnumValue table 
It should be terminated with a NULL entry, otherwise we just stray into the
realms of cryptographic libraries^W^W random memory usage.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3319>
 2022-11-04 17:59:21 +00:00
Edward Hervey 0213536f77 libav: avcfg: Avoid brittle comparision 
Subtracting a gint from another (or a guint from another) has no guarantees that
it will result in a gint.

Therefore do the actual comparision instead.

Also use the *actual* type for comparing flags (the field value types are different)

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3319>
 2022-11-04 17:59:21 +00:00
Edward Hervey cadc067bde libav: avcfg: Don't leak duplicate enumvalue entries 
If we discard the duplicates, we also need to free the allocated strings.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3319>
 2022-11-04 17:59:21 +00:00
Edward Hervey 97bfb8b6cb imagesequencesrc; Fix leaks 
* The path was leaked
* The custom buffer was never freed

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3319>
 2022-11-04 17:59:21 +00:00
Edward Hervey 6ffae88a9f qtdemux: Fix cenc-related leaks 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3319>
 2022-11-04 17:59:21 +00:00
Edward Hervey aa61662632 deinterlace: Don't leak metas 
There is no correlation between the frame being NULL and the metas not being
present.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3319>
 2022-11-04 17:59:21 +00:00
Edward Hervey 094d27585e validate: plug some leaks 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3319>
 2022-11-04 17:59:21 +00:00
Edward Hervey 7f86e925e2 adaptivedemux2: Fix collection leaks 
* The collection on the period was never unreffed
* The collection in the message handler was never unreffed

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3319>
 2022-11-04 17:59:21 +00:00
Edward Hervey 93d8b3f660 parsebin: Fix leak 
Release the reference on the collection which was given to us when extracting it
from the event/message.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3319>
 2022-11-04 17:59:21 +00:00
Jan Schmidt b2f2a9125c mssdemux2: Update for adaptivedemux2 refactoring 
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3314>
 2022-11-05 04:00:31 +11:00
Jan Schmidt 565f47b4f3 adaptivedemux2: Move stream_seek() to the Stream class 
Move the last stream specific vfunc from the demux
class to the stream class.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3314>
 2022-11-05 04:00:31 +11:00
Jan Schmidt 2fe641353d adaptivedemux2: Refactor stream methods into the stream 
Unlike the legacy elements, GstAdaptiveDemuxStream is a GObject now,
so a bunch of things that were actually stream methods on the
parent demux object can directly become stream methods now.

Move the stream class out to a header of its own.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3314>
 2022-11-05 04:00:31 +11:00
Jan Schmidt 9f89b8e3ef hlsdemux2/m3u8: Implement EXT-X-GAP parsing 
Read the EXT-X-GAP tag and set is_gap on the segment.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3314>
 2022-11-05 03:24:27 +11:00
Jan Schmidt 55db033570 hlsdemux2/m3u8: Refactor parsing for readability 
Small readability improvements in the parsing code

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3314>
 2022-11-05 03:24:27 +11:00
Jan Schmidt bfeb3c5625 adaptivedemux2/downloadhelper: Remove return val for download_request_add_buffer() 
The function can't actually fail, and the only caller
was ignoring the result anyway, so remove the return value.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3314>
 2022-11-05 03:24:27 +11:00
Jan Schmidt 0f8c38c190 adaptivedemux2/downloadhelper: Add debug output of response headers 
Dump the HTTP response headers at TRACE level

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3314>
 2022-11-05 03:24:27 +11:00
Jan Schmidt d1ae35f08a adaptivedemux2/downloadhelper: Don't mark transfer as complete/error if cancelled. 
If the state of the download request was reset to UNSENT,
it was cancelled. Don't update the state to COMPLETE or ERRORED
in on_read_ready().

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3314>
 2022-11-05 03:24:26 +11:00
Jan Schmidt 46517d5aca adaptivedemux2/downloadhelper: Ignore spurious read failure 
Sometimes g_input_stream_read_all_finish() can return
0 bytes, but still succeed (return TRUE) and have more
data available later. Only finish the transfer
if it returns 0 bytes *and* FALSE with no error.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3314>
 2022-11-05 03:24:26 +11:00