[Bug 788759] qtdemux: fix 'stsd' table leak and nested caps leak

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Sun Oct 22 10:32:32 UTC 2017


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

--- Comment #8 from Jun Xie <jun.xie at samsung.com> ---
(In reply to Sebastian Dröge (slomo) from comment #7)
> Review of attachment 361713 [details] [review]:
> 
> Don't change the permissions of files.
> 
> ::: gst/isomp4/qtdemux.c
> @@ +2543,3 @@
>      }
>    }
> +  g_free (stream->stsd_entries);
> 
> This will crash when called another time, might want to set it to 0 and the
> number of entries too.

thanks for reviewing, please review the new patch again.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list