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 751628 - tests: caps: test_intersect_flagset failure
tests: caps: test_intersect_flagset failure
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
unspecified
Other Linux
: Normal normal
: 1.5.90
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-06-29 04:57 UTC by Vineeth
Modified: 2015-08-16 13:41 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
fix segfaults in caps tests (1.13 KB, patch)
2015-06-29 04:58 UTC, Vineeth
committed Details | Review

Description Vineeth 2015-06-29 04:57:00 UTC
test_intersect_flagset fails because when caps is being 
 created, flags and mask are being casted to uint64,
 while the same are uint. This results in segfault as below
 gst/gstcaps.c:968:E:operations:test_intersect_flagset:0: (after this point) Received signal 11 (Segmentation fault)
Comment 1 Vineeth 2015-06-29 04:58:04 UTC
Created attachment 306261 [details] [review]
fix segfaults in caps tests
Comment 2 Tim-Philipp Müller 2015-06-29 12:48:27 UTC
Looks correct, the value collect signature for flagsets is "ii". Pushed, thanks for the patch:

commit 17300389535b30a2986805dba9254fcee613d36a
Author: Vineeth TM <vineeth.tm@samsung.com>
Date:   Mon Jun 29 13:57:11 2015 +0900

    tests: caps: fix test_intersect_flagset failure
    
    test_intersect_flagset fails because when caps is being
    created, flags and mask are being cast to uint64 while
    they should be uint. This results in invalid memory access
    or a segfault.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=751628
Comment 3 Jan Schmidt 2015-06-29 13:05:24 UTC
Oops - sorry that was from switching to 64-bit flagsets during dev, and failing to switch back to 32-bit in all the right places.