After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 770772 - glupload: 91fea30ff4 makes android tutorial 5 crash on Galaxy S6
glupload: 91fea30ff4 makes android tutorial 5 crash on Galaxy S6
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal blocker
: 1.9.90
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-09-02 22:23 UTC by Olivier Crête
Modified: 2016-10-31 14:31 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Olivier Crête 2016-09-02 22:23:08 UTC
This is a regression, it causes a segfault inside _gl_memory_upload_transform_caps ()
Comment 1 Sebastian Dröge (slomo) 2016-09-03 06:38:49 UTC
Do you have a backtrace? Where does it crash and why?
Comment 2 Matthew Waters (ystreet00) 2016-09-04 07:50:47 UTC
This is because the upload element removes and nullifies the upload object in _stop() but doesn't recreate it anywhere after the element's instance _init().
Comment 3 Tim-Philipp Müller 2016-09-12 19:13:13 UTC
Is this fixed by

commit c16d57cc9a64f1aeaa20ac079feb21f33893224d
Author: Alessandro Decina <alessandro.d@gmail.com>
Date:   Mon Sep 5 14:44:24 2016 +1000

    glupload: create the GstGLUpload object in ::transform_caps
    
    Previously it was created in the init function and destroyed in ::stop, which
    lead to segfaults when reusing the element.
    
    Now the upload object is created in ::transform_caps if it is NULL, which is the
    earliest we need it. The other vfuncs already bail out if the upload object is
    NULL, which means that negotiation wasn't done.


by any chance?
Comment 4 Alessandro Decina 2016-09-14 00:06:55 UTC
Yes