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 327871 - [videobox] crash when cropping
[videobox] crash when cropping
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other All
: Normal critical
: 0.10.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-01-20 14:40 UTC by Christoph Burghardt
Modified: 2006-01-20 15:10 UTC
See Also:
GNOME target: ---
GNOME version: 2.11/2.12


Attachments
proposed patch (547 bytes, patch)
2006-01-20 14:41 UTC, Christoph Burghardt
committed Details | Review

Description Christoph Burghardt 2006-01-20 14:40:55 UTC
Steps to reproduce:
in gstvideobox.c the oil_init is surrounded between HAVE_LIBOIL, but not the
oil_splat and oil_memcopy-functions in the rest of the source. so if HAVE_LIBOIL
is not defined (why, i don't know, but i have liboil installed, it compiled
correctly) the oil_init is omitted and gstvideobox does crash. because gstreamer
depends on liboil my proposed patch is to remove the #ifdef. that make things
work for me :)

Stack trace:


Other information:
Comment 1 Christoph Burghardt 2006-01-20 14:41:38 UTC
Created attachment 57732 [details] [review]
proposed patch
Comment 2 Tim-Philipp Müller 2006-01-20 15:10:21 UTC
Fixed in CVS, thanks for the patch! (doesn't happen when you test videobox with videotestsrc, because videotestsrc initialises liboil as well ...)

2006-01-20  Tim-Philipp Muller  <tim at centricular dot net>

        * gst/videobox/gstvideobox.c: (gst_video_box_class_init):
          Don't forget to initialize liboil, otherwise our oil functions
          will crash (fixes #327871; patch by: Christoph Burghardt
          <hawkes at web dot de>).