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 422295 - GstController crashes with quadratic and cubic interpolation modes
GstController crashes with quadratic and cubic interpolation modes
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other All
: Normal critical
: 0.10.13
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-03-24 14:37 UTC by Milosz Derezynski
Modified: 2007-06-08 20:45 UTC
See Also:
GNOME target: ---
GNOME version: 2.17/2.18


Attachments
Not a direct testcase, but this is the code we've encountered the problem within (6.29 KB, text/plain)
2007-03-24 14:39 UTC, Milosz Derezynski
Details

Description Milosz Derezynski 2007-03-24 14:37:56 UTC
Steps to reproduce:
1. Instantiate a GstController 
2. Set up intervals and an interpolation mode of either GST_INTERPOLATE_CUBIC or GST_INTERPOLATE_QUADRATIC
3. Crash


Stack trace:
Seems like CUBIC and QUADRATIC are not implemented at all, and lead to an application crash, they are however documented (!)

Other information:
Comment 1 Milosz Derezynski 2007-03-24 14:39:19 UTC
Created attachment 85221 [details]
Not a direct testcase, but this is the code we've encountered the problem within

This is a simple crossfader (pass file 1 and file 2 as args to the app), in which it's made pretty obvious that the controller issue exists.
Comment 2 Sebastian Dröge (slomo) 2007-03-24 17:22:41 UTC
Yes, they're simply not implemented yet and this should be noted in the documentation IMHO.

Implementing those modes is somewhere on my TODO list.
Comment 3 Milosz Derezynski 2007-03-24 23:02:03 UTC
Can you fix the docs for the next core release to mention that those modes are not implemented and map both to the linear interpolation impl in the code?

That way apps at least wouldn't crash heh. Or awesome would be if you could implement them for next core.
Comment 4 Tim-Philipp Müller 2007-05-04 12:41:30 UTC
 2007-05-04  Tim-Philipp Müller  <tim at centricular dot net>

        * libs/gst/controller/gstcontroller.c:
        (gst_controlled_property_set_interpolation_mode):
        * libs/gst/controller/gstinterpolation.c:
          Don't crash if someone tries to set an interpolation mode that
          is invalid or that isn't supported yet. Fixes #422295.

        * tests/check/libs/controller.c: (GST_START_TEST),
        (gst_controller_suite):
          Add a test case for the above.

 2007-05-04  Tim-Philipp Müller  <tim at centricular dot net>

        * tests/check/Makefile.am:
        * tests/check/elements/volume.c: (GST_START_TEST), (volume_suite):
          Minimal check for volume's GstController usability; also another
          test for #422295.

Comment 5 Sebastian Dröge (slomo) 2007-06-08 20:45:07 UTC
Cubic interpolation was added in CVS some days ago.