[gstreamer-bugs] [Bug 325504] New: flacdec gst_flac_dec_convert_src [mis]uses g_assert

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=325504
 GStreamer | gst-plugins-good | Ver: HEAD CVS

           Summary: flacdec gst_flac_dec_convert_src [mis]uses g_assert
           Product: GStreamer
           Version: HEAD CVS
          Platform: Other
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: alessandro at nnva.org
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: 2.13/2.14
   GNOME milestone: Unspecified


Please describe the problem:
gst_flac_dec_convert_src uses g_assert to assert conditions which can be, and
actually are, better handled by simply returning FALSE.

Steps to reproduce:
Using rhythmbox (cvs) with gstreamer (cvs) to play flac files.
It seems like rhythmbox does the first query_position before the first frame
has
been decoded.



Actual results:
Rhythmbox is aborted

Expected results:


Does this happen every time?


Other information:


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.