[Bug 681459] New: gstrtmpsink: memory leak

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=681459
  GStreamer | gst-plugins-bad | git

           Summary: gstrtmpsink: memory leak
    Classification: Platform
           Product: GStreamer
           Version: git
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: dregade at viewsurf.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


Created an attachment (id=220696)
 View: https://bugzilla.gnome.org/attachment.cgi?id=220696
 Review: https://bugzilla.gnome.org/review?bug=681459&attachment=220696

patvh against RTMP_ParseURL memoryleak

Hi,

In gst_rtmp_sink_uri_set_uri, a test is performed in order to be sure uri is
correct for librtmp. This test calls RTMP_ParseURL with 3 AVal pointers as
parameters: host, playpath and app.

AVal is a struct with a char* + int. After RTMP_ParseURL call, host.av_val and
app.av_val both refer a substring of "uri". But playpath.av_val may be the
result of a malloc so it needs to be freed.

I attached a patch.

Regards

-- 
Configure bugmail: https://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.