[Bug 737771] New: souphttpclientsink: Stream header buffer lifetime assumptions are incorrect

[GStreamer (bugzilla.gnome.org)]

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

           Summary: souphttpclientsink: Stream header buffer lifetime
                    assumptions are incorrect
    Classification: Platform
           Product: GStreamer
           Version: git
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: nirbheek.chauhan at gmail.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


Stream headers are updated whenever ::set_caps is called, so we can't assume
they'll be valid till the message body is written out. We *can* assume that for
queued buffers, but SOUP_MEMORY_STATIC is still wrong for those. 

This false assumption often leads to corrupted stream headers being sent out if
a pipeline containing souphttpclientsink is restarted.

The attached patch fixes this.

-- 
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.