[gstreamer-bugs] [Bug 364388] New: size variable is not always initialized in gst_base_src_start()

[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=364388

  GStreamer | gstreamer (core) | Ver: HEAD CVS

           Summary: size variable is not always initialized in
                    gst_base_src_start()
           Product: GStreamer
           Version: HEAD CVS
          Platform: Other
        OS/Version: All
            Status: UNCONFIRMED
          Severity: trivial
          Priority: Normal
         Component: gstreamer (core)
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: sergey.scobich at gmail.com
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


Hi, it seems that in gstbasesrc.c in gst_base_src_start() the variable size is
not always initialized and used later.
It happends when basesrc->segment.format != GST_FORMAT_BYTES for ex
GST_FORMAT_TIME in my case.
later size is used here:

/* run typefind if we are random_access and the typefinding is enabled. */
if (basesrc->random_access && basesrc->data.ABI.typefind && size != -1) {

may be it is better to initialize size with -1 from the start.


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email