summaryrefslogtreecommitdiff
path: root/ext
Commit message (Collapse)AuthorAgeFilesLines
* curlsink: catch an unknown errorPatricia Muscalu2016-04-061-0/+9
| | | | | | | | | | In this case the socket callback has not been called by libcurl and the curlsink has not been notified about any connection problems by libcurl. This indicates that it's a bug in libcurl so catch it as an unknown error. https://bugzilla.gnome.org/show_bug.cgi?id=754432
* wayland: fix null pointer dereference on errorMatthew Waters2016-04-061-1/+2
| | | | | | | gnome-shell doesn't the support wl_scaler interface which makes creating a wayland display fail creation. Found in the generic state changes test
* qml: Fix leak of the OpenGL contextsSergey Borovkov2016-04-061-1/+4
| | | | | | [Matthew Waters]: add NULL checks before unreffing https://bugzilla.gnome.org/show_bug.cgi?id=762999
* dashdemux: correctly handle an HTTP-XSDATE that is exactly the size of the ↵Florin Apostol2016-04-061-6/+8
| | | | | | | | | | | | | | | | date string The code in the gst_dash_demux_parse_http_xsdate() was trying to handle the case where the string is not null terminated by resizing the buffer and appending a zero byte. This does not work if the buffer is exactly the length of the string because the gst_buffer_resize() function does not re-allocate the buffer, it just changes its size. If a buffer is passed to gst_dash_demux_parse_http_xsdate() that is exactly the length of the string, the function fails with an assert failure in gst_buffer_resize(). https://bugzilla.gnome.org/show_bug.cgi?id=762148
* curlbasesink: don't send empty buffersDavid Fernandez2016-04-061-2/+7
| | | | | | Fixes problem in curlhttpsink when qtmux uses faststart. https://bugzilla.gnome.org/show_bug.cgi?id=762013
* mssdemux: do not assert on fragment reloads for non-liveThiago Santos2016-04-061-2/+0
| | | | | It can be used as a resource to verify if the server has updated something in the Manifest when downloads are failing
* opusdec: fix wrong buffer being checked for missing dataVincent Penquerc'h2016-04-061-2/+2
| | | | | | | This caused a decoding error if the resulting (wrong) buffer size was passed to the Opus decoding API. https://bugzilla.gnome.org/show_bug.cgi?id=758158
* mpeg2enc: Provide format as a string instead of a char to gst_structure_newVivia Nikolaidou2016-03-041-1/+1
| | | | The format was provided as 'I420' instead of "I420", causing a crash.
* ladspa: Fix some debugsThibault Saunier2016-02-171-4/+3
|
* gtk(gl)sink: remove the signal handlers on finalizeMatthew Waters2016-02-162-3/+12
| | | | | | | | | | | It's possible that the sink element will be freed before the widget is destroyed. When the widget was eventually destroyed, it was attempting to access member variables of the freed sink struct which resulted in undefined behaviour. Fix by disconnecting our signal on finalize. https://bugzilla.gnome.org/show_bug.cgi?id=762098
* glvideomixer: fix checker vbo leakWang Xin-yu (王昕宇)2016-02-011-0/+5
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=760925
* gleffects: fix gleffects fisheye shader compile errorHaihua Hu2016-01-291-1/+1
| | | | | | | | On some embedded systems, sqrt() is not supported in the shader, use the actual value of sqrt(2) instead. Signed-off-by: Haihua Hu <b55597@freescale.com> Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=761271
* sbc: sbcdec: Fix frame length calculationTim Sheridan2016-01-161-4/+4
| | | | | | | | | 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. https://bugzilla.gnome.org/show_bug.cgi?id=742446
* qml: Mark material dirty when texture buffer is updatedSergey Borovkov2016-01-151-0/+1
| | | | | | Qt might not redraw the scene otherwise. https://bugzilla.gnome.org/show_bug.cgi?id=758286
* openjpegdec: Don't crash when decoding returns NULL data for any componentSebastian Dröge2015-12-181-0/+9
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=758943
* gltransformation: clear to transparentMatthew Waters2015-12-121-1/+1
| | | | | Otherwise composition will result in a black frame outside the transformed video.
* openjpegenc: All frames in JPEG2000 are keyframes, mark them as suchSebastian Dröge2015-12-081-0/+1
|
* qml: reuse existing GstQSGTextureRoman Nowicki2015-11-201-4/+2
| | | | | | Fixes a memory leak leaking the texture objects. https://bugzilla.gnome.org/show_bug.cgi?id=758286
* qml: activate the wrapped context when bindingMatthew Waters2015-11-201-4/+9
| | | | | | Mitigates the following critical gst_gl_context_thread_add: assertion 'context->priv->active_thread == g_thread_self ()' failed
* qml: proper initialization if scene is already initializedRoman Nowicki2015-11-201-1/+5
| | | | | | | | The scene graph can be initialized when the we receive window handle change notification and so we will not receive a scenegraph initialization notification. Initialize ourself in this case. https://bugzilla.gnome.org/show_bug.cgi?id=758337
* glimagesink: Fix structure memory leakVineeth TM2015-11-171-1/+3
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=758205
* gtk: add the overlaycomposition feature to the template capsMatthew Waters2015-11-101-2/+5
| | | | | | | | There is a possibility that the _get_caps impl will be called with the feature in the filter caps which when interecting with the template, will return EMPTY and therefore fail negotiation. https://bugzilla.gnome.org/show_bug.cgi?id=757854
* assrender: don't send flush events on seeksTim-Philipp Müller2015-11-071-45/+7
| | | | | | | | | | It's for the upstream element driving the pipeline to handle seeks and send flush events or not, filters should not interfere here, otherwise downstream pads could be flushing before upstream pads are flushing, which can result in GST_FLOW_ERROR being sent instead of GST_FLOW_FLUSHING when trying to forward sticky events at just the wrong moment.
* opusenc: Place 48kHz first in the capsSebastian Dröge2015-11-031-1/+6
| | | | For all the other sample rates the encoder will have to resample internally.
* opusdec: Assume 48kHz if no sample rate is given in the headerSebastian Dröge2015-11-031-0/+2
|
* waylandsink: don't advertise the local wl_display contextGeorge Kiagiadakis2015-10-261-6/+0
| | | | | | | | | | | | | | | | This wl_display proxy is temporary only until waylandsink goes NULL, at which point the connection to the display is disposed. Unfortunately, if this is advertised as a GstContext, playbin will cache it and re-feed it to the sink when it goes PLAYING again, but the wl_display pointer will at that point be invalid and cause a crash. Another solution to the problem would be to also cache the GstWlDisplay object inside the GstContext, which would automatically ref-count the display connection, but I see no reason in doing that at the moment, as there are no known users of this GstContext outside waylandsink. It's probably better to avoid chasing hidden refcounts. https://bugzilla.gnome.org/show_bug.cgi?id=756567
* hlsdemux: don't crash or leak memory on broken master playlistAndré Draszik2015-10-251-3/+4
| | | | | | | | | | | | | | | | | If a (master) playlist contains a variant list entry without a URI then during parsing of the next variant list entry we are a) leaking the entry we're currently parsing (new_list), and b) free'ing the pointer to the previous list entry (list) without updating the pointer. Hence when then adding the URI for the latest parsed entry, incorrect information is stored, as the information is used from 'list' which is not valid memory anymore, also leading to crashes. Fix this by correctly storing the new variant list entry pointer as needed. https://bugzilla.gnome.org/show_bug.cgi?id=756861
* srtp: Fix critical warning trying to retrieve SSRC from a non fb messageSantiago Carot-Nemesio2015-10-231-1/+0
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=756876
* srtpdec: skip padding when mapping RTP packetsMiguel París Díaz2015-10-201-2/+4
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=756653
* mpdparser: Set default last_byte_pos to -1Sebastian Dröge2015-10-201-1/+1
| | | | | | | The value is optional in the range, and if it is absent it means we should download until the end of stream. Not until position 0. https://bugzilla.gnome.org/show_bug.cgi?id=748316
* hls/m3u8: Update current position in all casesEdward Hervey2015-10-202-15/+25
| | | | | | | | | | In order to ensure the sequence_position will always be consistently updated, store the current file duration. This way, when we advance, we can always increment the position based on what was previously outputted. https://bugzilla.gnome.org/show_bug.cgi?id=752132
* gtk: separate out the widget/window destroy callbacksMatthew Waters2015-10-191-7/+10
| | | | | | | Fixes assertion due to the sink_finalize() being run before the widget destroy callback. https://bugzilla.gnome.org/show_bug.cgi?id=755969
* dtls: Fix name conflict with openssl on win32Paul Arzelier2015-10-171-0/+7
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=756592
* hlsdemux: Avoid negative sequence numbersEdward Hervey2015-10-151-4/+12
| | | | | | | | | | | | | | | For live streams, we want to make sure there's a certain distance between the sequence to play and the last (earliest) fragment. The problem is that it assumes there are at least 3 fragments in the playlist, which might not always be the case (like in the case of a server restarting and gradually adding fragments). In order to avoid ending up with negative sequence numbers (which will just loop forever), limit the new target sequence number to the highest of: * either the first sequence number of the playlist (fallback) * or 3 fragments from the last one (standard behaviour)
* dashdemux: fix undefined reference to gst_ntp_clock_newJulien Isorce2015-10-111-1/+1
| | | | | | Found with gst-inspect-1.0 -b in gst-uninstalled env. https://bugzilla.gnome.org/show_bug.cgi?id=756383
* glvideomixer: Proxy the ignore-eos videoaggregator property as wellNirbheek Chauhan2015-10-071-0/+7
| | | | Identical to how the z-order property is proxied
* dashdemux: fixed has_next_periodFlorin Apostol2015-10-071-0/+8
| | | | | | | gst_mpd_client_has_next_period now calls gst_mpd_client_setup_media_presentation to refresh the period information. https://bugzilla.gnome.org/show_bug.cgi?id=756186
* rtmpsrc: plug memory-leaksHavard Graff2015-10-051-4/+4
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=756001
* dtls: fix printf format on win32Polochon_street2015-10-051-1/+1
|
* dtls: fix printf format on win32Paul Arzelier2015-10-051-2/+2
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=755981
* dashdemux: Correctly assign earliest pts instead of overriding it with first ↵John Chang2015-10-051-1/+1
| | | | | | offset https://bugzilla.gnome.org/show_bug.cgi?id=755824
* dashdemux: Implement lazy-loading of external periodsSebastian Dröge2015-10-023-9/+58
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=752230
* mpdparser: Load OnLoad external resources immediately instead of on demandSebastian Dröge2015-10-021-0/+172
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=752230
* mpdparser: Add support for loading external SegmentListsSebastian Dröge2015-10-021-9/+143
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=752230
* mpdparser: Add support for loading external AdaptationSetsSebastian Dröge2015-10-021-3/+149
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=752230
* mpdparser: Implement loading of external Period nodesSebastian Dröge2015-10-021-13/+190
| | | | | | | | | | | The same has to be done for AdaptationSet and SegmentList nodes still. Also this does not correctly implement the semantics: by default Period (and other nodes) should only be loaded when needed, not in the very beginning. We need to implement lazy loading for them, which means adjusting gst_mpd_client_setup_media_presentation(). https://bugzilla.gnome.org/show_bug.cgi?id=752230
* mpdparser: Store an URI downloader in the parser for downloading additional ↵Sebastian Dröge2015-10-023-0/+21
| | | | | | MPD resources if needed https://bugzilla.gnome.org/show_bug.cgi?id=752230
* mpdparser: Parse xlink attributes from Period, AdaptationSet and SegmentListSebastian Dröge2015-10-022-0/+78
| | | | | | | We still have to do something useful with them, like actually loading the content behind the URL. https://bugzilla.gnome.org/show_bug.cgi?id=752230
* dashdemux: Properly handle relative and non-HTTP URIs for the headers/indicesSebastian Dröge2015-10-021-7/+3
| | | | | | | gst_uri_join_strings() will return the second parameter if it is an absolute URI. No need to do a (wrong) check if the URI is absolute or not beforehand. https://bugzilla.gnome.org/show_bug.cgi?id=755134
* mpdparser: If no Initialization is present in the SegmentBase, assume it is ↵Sebastian Dröge2015-10-021-26/+26
| | | | | | before the indexRange https://bugzilla.gnome.org/show_bug.cgi?id=755132