[Bug 737771] 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

Sebastian Dröge (slomo) <slomo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
                 CC|                            |slomo at coaxion.net
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |1.5.1

--- Comment #2 from Sebastian Dröge (slomo) <slomo at coaxion.net> 2014-10-02 09:51:31 UTC ---
commit f35f3ccf7c9d659c070a636dbf62ce7d4127053b
Author: Nirbheek Chauhan <nirbheek at centricular.com>
Date:   Thu Oct 2 14:26:08 2014 +0530

    souphttpclientsink: Fix lifetime of stream headers and queued buffers

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

    Also, add some debug logging for stream header interactions.

    https://bugzilla.gnome.org/show_bug.cgi?id=737771

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