summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Release 1.15.21.15.2Tim-Philipp Müller2019-02-266-102/+1241
|
* Update docsTim-Philipp Müller2019-02-26129-142/+161
|
* Update translationsTim-Philipp Müller2019-02-2641-313/+135
|
* msdkdec: release the occupied surface for MFX_WRN_DEVICE_BUSYHaihao Xiang2019-02-241-0/+6
| | | | | | | | | When MFXVideoDECODE_DecodeFrameAsync () returns MFX_WRN_DEVICE_BUSY with an output surface, a new input surface is required when retrying MFXVideoDECODE_DecodeFrameAsync (). This fixes the out-of-surface issue mentioned in https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/890
* msdk: don't destroy an image twiceHaihao Xiang2019-02-241-2/+19
| | | | | | | | | | | This gets rid of annoying message in the log, e.g. run the pipeline below: gst-launch-1.0 videotestsrc num-buffers=100 ! \ video/x-raw,format=NV12,width=352,height=288 ! msdkh264enc ! filesink \ location=test.h264 [LIBVA]:CRITICAL - DdiMedia_DestroyImage:4357: Invalid image
* msdk: move the comment to the correct placeHaihao Xiang2019-02-241-5/+6
|
* srtobject: check for null local address to avoid SIGSEVAaron Boxer2019-02-241-4/+5
|
* ccconverter: fix typo in framerateMathieu Duponchelle2019-02-211-1/+1
|
* audiobuffersplit: Added max-silence-time propertyVivia Nikolaidou2019-02-212-82/+117
|
* mpegtsmux: allow specifying the PID of the PCR streamMathieu Duponchelle2019-02-201-5/+21
| | | | | The structure passed through the prog-map can now contain a PCR_<prog_id>=sink_<PID> key-value pair.
* decklink: add Hardware tag to element metadataTim-Philipp Müller2019-02-194-4/+8
|
* wasapi: add Hardware tag to element metadataTim-Philipp Müller2019-02-192-2/+2
|
* winks: add Hardware tag to element metadataTim-Philipp Müller2019-02-191-1/+1
|
* androidmedia: add Hardware tag to element metadataTim-Philipp Müller2019-02-193-3/+3
|
* applemedia: add Hardware tag to element metadataTim-Philipp Müller2019-02-194-5/+5
|
* msdk: add Hardware tag to element metadataTim-Philipp Müller2019-02-1912-12/+12
|
* nvenc, nvdec: add Hardware tag to element metadataTim-Philipp Müller2019-02-193-3/+3
|
* curlhttpsrc: set BUFFER_OFFSET when creating GstBufferAlex Ashley2019-02-191-0/+1
| | | | | | | To make curlhttpsrc behave more like souphttpsrc, set the BUFFER_OFFSET in its output buffers to match the segment start. This means that in a HTTP RANGE request, the BUFFER_OFFSET will match the value in the RANGE request.
* curlhttpsrc: export same HTTP error status as souphttpsrcAlex Ashley2019-02-192-4/+23
| | | | | | | To make it closer to a drop-in replacement for souphttpsrc, expose the same gst_error_message_with_details as souphttpsrc, so that applications can received the HTTP status code and reason when an error occurs.
* curlhttpsrc: fix various leaks and thread safety issuesAlex Ashley2019-02-195-230/+303
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | curlhttpsrc uses a single thread running the gst_curl_http_src_curl_multi_loop() function to handle receiving data and messages from libcurl. Each instance of curlhttpsrc adds an entry into a queue in GstCurlHttpSrcMultiTaskContext and waits for the multi_loop to perform the HTTP request. Valgrind has shown up race conditions and memory leaks: 1. gst_curl_http_src_change_state() does not wait for the multi_loop to complete before going to the NULL state, which means that an instance of GstCurlHttpSrc can be released while gst_curl_http_src_curl_multi_loop() still has a reference to it. 2. if multiple elements try to be removed from the queue at once, only the last one is deleted. 3. source->caps is leaked 4. curl multi_handle is leaked 5. leak of curl_handle if URI not set 6. leak of http_headers when reusing element 7. null pointer dereference in negotiate caps 8. double-free of the default user-agent string 9. leak of multi_task_context.task This commit changes the logic so that each element has a connection status, which is used by the multi_loop to decide when to remove an element from its queue. An instance of curlhttpsrc will not enter the NULL state until its reference has been removed from the queue. When shutting down the curl multi loop, the memory allocated from the call to curl_multi_init() is now released. When gstadaptivedemux uses a URI source element, it will re-use it for multiple requests, moving it between READY and PLAYING between each request. curlhttpsrc was leaking the http_headers structure in this use case. The gst_curl_http_src_negotiate_caps() function extracts the "response-headers" field from the http_headers, but did not check that this field might be NULL. If the user-agent property is set, the global user-agent string was freed. This caused a double-free error if the user-agent is ever set a second time during the execution of the process. There are situations within curlhttpsrc where the code needs both the global multi_task_context mutex and the per-element buffer_mutex. To avoid deadlocks, it is vital that the order in which these are requested is always the same. This commit modifies the locking order to always be in the order: 1. multi_task_context.task_rec_mutex 2. buffer_mutex Fixes #876
* tests: curlhttpsrc: add unit testsAlex Ashley2019-02-193-0/+698
| | | | | | | | | | | | | | | | | Based upon the souphttpsrc tests, add unit tests for the curlhttpsrc element. The souphttpsrc tests are able to use an HTTP server that is provided as part of the soup library. This does not exist in the curl library, therefore these tests provide a very simple HTTP server using the GIO library. These curlhttpsrc tests contain one new test that does not come from the souphttpsrc tests. The test_multiple_http_requests test tries to reproduce the way in which GstAdaptiveDemux makes use of URI source elements. GstAdaptiveDemux creates a bin with the httpsrc element and a queue element and sets the locked state of that bin to TRUE, so that it does not follow the state transitions of its parent. It then moves this bin to the PLAYING state to start each download and back to READY when the download completes.
* msdkvpp: allow UYVY in DMABuf mode in the sink padHaihao Xiang2019-02-191-1/+1
| | | | | | | | Tested with the pipeline below: gst-launch-1.0 videotestsrc num-buffers=1 ! msdkvpp ! \ video/x-raw\(memory:DMABuf\),format=UYVY ! msdkvpp ! video/x-raw, \ format=YUY2 ! filesink location=a.yuv
* msdkvpp: allow UYVY in the src padHaihao Xiang2019-02-191-2/+2
| | | | | | | | | | | | | This make the pipeline below works: gst-launch-1.0 videotestsrc num-buffers=1 ! msdkvpp ! \ video/x-raw,format=UYVY ! filesink location=a.yuv Once https://github.com/intel/media-driver/pull/526 in the media-driver is merged, the pipeline below also works: gst-launch-1.0 videotestsrc num-buffers=1 ! msdkvpp ! \ video/x-raw\(memory:DMABuf\),format=UYVY ! filesink location=a.yuv
* msdk: map GST_VIDEO_FORMAT_UYVY to VA_FOURCC_UYVYHaihao Xiang2019-02-191-0/+4
|
* msdkvpp: allow BGRx in DMABuf mode in the src padHaihao Xiang2019-02-191-1/+1
| | | | | | | Tested with the pipeline below: gst-launch-1.0 videotestsrc ! msdkvpp ! \ video/x-raw\(memory:DMABuf\),format=BGRx ! glimagesink
* vcdsrc: Remove unusable VCD sourceBastien Nocera2019-02-1812-757/+3
| | | | | | | | | | | | | | | | | | The VCD source was ported in 2014 (commit 89eb1e9), but the necessary "cdxaparse" plugin, which is used to "Parse a .dat file (VCD) into raw mpeg1" was never ported. This means that the probable main user for the feature, totem, hasn't actually been able to play back VCDs, since 2012, when it switched to using GStreamer 1.0. Note that even if cdxaparse was finally ported, a lot of work would still be necessary before it is considered usable. Notably, it is missing disc image support [1] and some VCDs just cannot be opened for reading [2]. [1]: https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/898 [2]: https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/899
* closedcaption: Use portable variadic macro syntaxNirbheek Chauhan2019-02-151-0/+21
| | | | | | args... is not understood by MSVC: error C2010: '.': unexpected in macro parameter list
* closedcaption: Port plugin to MSVCNirbheek Chauhan2019-02-152-20/+20
| | | | | pthread is not portable, so we can't use a pthread mutex use GMutex instead.
* waylandsink: Don't create throwaway empty regionsDaniel Stone2019-02-151-8/+2
| | | | | | | | Instead of creating a region, adding nothing to it, setting that as the input region and destroying the region, you can instead just pass NULL to wl_surface_set_input_region for the same effect. Fixes #702
* webrtc: fix rtx + bundleMatthew Waters2019-02-151-71/+85
| | | | | | | | If bundle was used in combination with rtx, only the bundled transport stream would have correctly configured rtx parameters. Iterate over the payloads upfront in the bundled case to ensure the correct payload mapping is set for the RTX elements.
* dtls: Don't abort on non-fatal issuesEdward Hervey2019-02-131-3/+6
| | | | | | | | OpenSSL will take care of returning valid context if there are only non-fatal issues. Don't abort in those cases and instead just print out the issues Fixes https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/811
* fdkaacenc: Fix draining with libfdk-aac v2.0.0Jan Alexander Steffens (heftig)2019-02-131-8/+13
| | | | | This release requires all buffer descriptor pointers to be valid, even when we provide zero input buffers.
* tsdemux: Skew correction should use the upstream DTSJan Schmidt2019-02-131-3/+6
| | | | | | | The MPEG-TS packetiser should use the upstream DTS for skew correction when running in that mode, as the DTS carries the upstream arrival time. The PTS (if it's set at all) is less useful, and can be invalid.
* ccconverter: Don't output empty buffersSebastian Dröge2019-02-121-1/+2
|
* decklinkvideosink: Use S334_EIA_608 instead of S334_EIA_708 for CEA608 outputSebastian Dröge2019-02-121-4/+4
|
* decklinkvideosink: S334 1A CEA608 closed captions have the first bit set for ↵Sebastian Dröge2019-02-121-3/+2
| | | | | | the first field And not for the second field. The logic was inverted here.
* msdkdec: set decode_only for output onlyHaihao Xiang2019-02-121-2/+5
| | | | | | MSDK may return MFX_ERR_MORE_DATA but without output surface Fixes https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/887
* msdk: verify the driver nameHaihao Xiang2019-02-121-1/+2
| | | | | | | We need to make sure the Intel graphics card is used in a dual GPU system. Fixes https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/895
* srt: re-enable passphraseJustin Kim2019-02-113-23/+51
| | | | | | | | With refactoring, supporting passphrase was removed accidently. This commit re-enables srt encryption and validates 'passphrase' by checking the return value of 'srt_setsockopt'. fix: #694
* openh264dec: Fix build with msvcSeungha Yang2019-02-111-6/+7
| | | | | | | Fix following build error ../subprojects/gst-plugins-bad/ext/openh264/gstopenh264dec.cpp(76): error C2121: Note that msvc usually complains #if inside macro
* Add device provider for AVFoundation capture devices.Josh Matthews2019-02-087-103/+443
|
* misc: Fix warnings on Cerbero MinGWNirbheek Chauhan2019-02-067-8/+15
| | | | | | | | | | | | | | | | | | gstladspa.c:360:5: error: zero-length ms_printf format string [-Werror=format-zero-length] vad_private.c:108:3: error: this decimal constant is unsigned only in ISO C90 [-Werror] gstdecklinkvideosink.cpp:478:32: error: comparison between 'BMDTimecodeFormat {aka enum _BMDTimecodeFormat}' and 'enum GstDecklinkTimecodeFormat' [-Werror=enum-compare] win/DeckLinkAPI_i.c:72:8: error: extra tokens at end of #endif directive [-Werror] win/DeckLinkAPIDispatch.cpp:35:10: error: unused variable 'res' [-Werror=unused-variable] gstwasapiutil.c:733:3: error: format '%x' expects argument of type 'unsigned int', but argument 8 has type 'DWORD' [-Werror=format] gstwasapiutil.c:733:3: error: format '%x' expects argument of type 'unsigned int', but argument 9 has type 'guint64' [-Werror=format] kshelpers.c:446:3: error: missing braces around initializer [-Werror=missing-braces] kshelpers.c:446:3: error: (near initialization for 'known_property_sets[0].guid.Data4') [-Werror=missing-braces]
* msdkdec: Release occupied surface for MFX_ERR_MORE_DATAHaihao Xiang2019-02-032-2/+14
| | | | | | | | | | | | | | | | | | | | An output surface is returned but without sync point when when MFXVideoDECODE_DecodeFrameAsync () returns MFX_ERR_MORE_DATA, this surface should be released too, otherwise the surface is occupied and it is easy to exhaust all pre-allocated mfx surfaces. Example pipeline (input_vp8.webm contains lots of frame with show_frame set to 0): gst-launch-1.0 filesrc location=input_vp8.webm ! matroskademux ! msdkvp8dec ! msdkvpp ! fakesink 0:00:05.995959693 19866 0x563f30f14590 ERROR default gstmsdkvideomemory.c:77:gst_msdk_video_allocator_get_surface: failed to get surface available ERROR: from element /GstPipeline:pipeline0/GstMatroskaDemux:matroskademux0: Internal data stream error.
* opencv cameraundistort: use G_GNUC_UNUSED for better portabilityNicola Murino2019-02-021-5/+3
|
* opencv: remove deprecated headersNicola Murino2019-02-022-10/+0
| | | | are not needed anymore
* opencv: port to c++Nicola Murino2019-02-0245-1316/+861
|
* msdkdec: explain the the extra ref of input bufferVíctor Manuel Jáquez Leal2019-02-021-0/+7
|
* msdkdec: avoid releasing the input buffer when it is still in useHaihao Xiang2019-02-021-5/+12
| | | | | | | | | The input buffer is released in gst_msdkdec_finish_task () when decoding some special clips however this buffer is still in use, so ref the input buffer before gst_msdkdec_finish_task () and unref it at the end of gst_msdkdec_handle_frame (). Fixes https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/862
* meson: orc-test is not requiredNirbheek Chauhan2019-01-311-3/+2
| | | | This is especially never available on iOS.
* msdkvp9dec: output P010_10LE buffer if possibleHaihao Xiang2019-01-311-2/+2
|
View Lorry Controller Status