[FFmpeg-devel] [PATCH 1/3] libavformat/libopenmpt: Fix memory leak in error path in read_header_openmpt().

2016-07-17 Thread Jörn Heusipp
Signed-off-by: Jörn Heusipp --- libavformat/libopenmpt.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavformat/libopenmpt.c b/libavformat/libopenmpt.c index fd38828..58a02be 100644 --- a/libavformat/libopenmpt.c +++ b/libavformat/libopenmpt.c @@ -89,8 +89,11 @@ stat

Re: [FFmpeg-devel] [PATCH 1/3] libavformat/libopenmpt: Fix memory leak in error path in read_header_openmpt().

2016-07-17 Thread Nicolas George
Le decadi 30 messidor, an CCXXIV, Jörn Heusipp a écrit : > Signed-off-by: Jörn Heusipp > --- > libavformat/libopenmpt.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) I am not objecting to the patch, but I think it shows demuxers would benefit from a deinit() method just like Rodge

Re: [FFmpeg-devel] [PATCH 1/3] libavformat/libopenmpt: Fix memory leak in error path in read_header_openmpt().

2016-07-17 Thread Michael Niedermayer
On Sun, Jul 17, 2016 at 03:37:12PM +0200, Jörn Heusipp wrote: > Signed-off-by: Jörn Heusipp > --- > libavformat/libopenmpt.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) applied btw, there is noone listed in MAINTAINERs for libavformat/libopenmpt.c can one of the authors/contrib