Re: [FFmpeg-devel] [PATCH] libavformat/ftp: Do not leak memory in routine ftp_features

2014-08-21 Thread Alexander Strasser
On 2014-08-20 06:27 +0200, Alexander Strasser wrote: > On 2014-08-20 01:25 +0200, Alexander Strasser wrote: > > Should fix CID1231988 (RESOURCE_LEAK) > > > > Signed-off-by: Alexander Strasser > > --- > > > > WARNING: Sorry, I only compile-tested so far. > > > > There is one remaining thing, I

Re: [FFmpeg-devel] [PATCH] libavformat/ftp: Do not leak memory in routine ftp_features

2014-08-19 Thread Alexander Strasser
On 2014-08-20 01:25 +0200, Alexander Strasser wrote: > Should fix CID1231988 (RESOURCE_LEAK) > > Signed-off-by: Alexander Strasser > --- > > WARNING: Sorry, I only compile-tested so far. > > There is one remaining thing, I am not sure of: > It looks like the variable feet could be uninitializ

[FFmpeg-devel] [PATCH] libavformat/ftp: Do not leak memory in routine ftp_features

2014-08-19 Thread Alexander Strasser
Should fix CID1231988 (RESOURCE_LEAK) Signed-off-by: Alexander Strasser --- WARNING: Sorry, I only compile-tested so far. There is one remaining thing, I am not sure of: It looks like the variable feet could be uninitialized at the point av_freep is called. I think it cannot, but I had to fol