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 737568 - video lib conversion in gstvideoaggregator.c leads glvideomixer problem
video lib conversion in gstvideoaggregator.c leads glvideomixer problem
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 1.5.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-09-29 08:52 UTC by comicfans44
Modified: 2014-11-08 14:15 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description comicfans44 2014-09-29 08:52:51 UTC
I found after commit  35bd1969f912cecda0fb5df9595a005e6ddf4e7a
 video: use video lib conversion code instead of copy

videotestsrc ! glvideomixer ! glimagesink 

gives error as :
** (gst-launch-1.0:25799): CRITICAL **: gst_video_converter_new: assertion 'in_info->width == out_info->width' failed

debug shows in 
gst_videoaggregator_update_converters

best_info is width = 1,  height = 1,  

seems this cause the assert.
(before this commit the best_info is also width=1,height=1,but maybe there's no assert check.)


backtrace as follows:
  • #4 gst_video_converter_new
    at ../../../../gst-libs/gst/video/video-converter.c line 126
  • #5 gst_videoaggregator_update_converters
    at ../../../../gst-libs/gst/video/gstvideoaggregator.c line 449
  • #6 gst_videoaggregator_pad_sink_setcaps
    at ../../../../gst-libs/gst/video/gstvideoaggregator.c line 684
  • #7 gst_videoaggregator_sink_event
    at ../../../../gst-libs/gst/video/gstvideoaggregator.c line 1613
  • #8 pad_event_func
    at ../../../../gst-libs/gst/base/gstaggregator.c line 1240
  • #9 gst_pad_send_event_unchecked
    at ../../gst/gstpad.c line 5155
  • #10 gst_pad_push_event_unchecked
    at ../../gst/gstpad.c line 4842
  • #11 push_sticky
    at ../../gst/gstpad.c line 3443
  • #12 events_foreach
    at ../../gst/gstpad.c line 570
  • #13 check_sticky
    at ../../gst/gstpad.c line 3499
  • #14 gst_pad_push_event
    at ../../gst/gstpad.c line 4967
  • #15 gst_base_src_set_caps
    at ../../../../libs/gst/base/gstbasesrc.c line 918
  • #16 gst_base_src_default_negotiate
    at ../../../../libs/gst/base/gstbasesrc.c line 3215
  • #17 gst_base_src_negotiate
    at ../../../../libs/gst/base/gstbasesrc.c line 3255
  • #18 gst_base_src_loop
    at ../../../../libs/gst/base/gstbasesrc.c line 2682
  • #19 gst_task_func
    at ../../gst/gsttask.c line 317
  • #20 default_func
    at ../../gst/gsttaskpool.c line 68

Comment 1 Sebastian Dröge (slomo) 2014-09-29 08:55:59 UTC
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.

http://cgit.freedesktop.org/gstreamer/gst-plugins-bad/commit/?id=2043def155030340fa1d2714d26817603c2c9d80