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

2014-08-25 Thread Alexander Strasser
On 2014-08-21 23:02 +0200, Alexander Strasser wrote: > Setting the pointer to NULL inside both ftp_send_command > and ftp_features is redundant. Generally always setting to > NULL in ftp_send_command seems safer, but throughout the file > that parameter was always passed initialized. So I do it her

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

2014-08-21 Thread Alexander Strasser
Setting the pointer to NULL inside both ftp_send_command and ftp_features is redundant. Generally always setting to NULL in ftp_send_command seems safer, but throughout the file that parameter was always passed initialized. So I do it here too for consistency. Should fix CID1231988 (RESOURCE_LEAK)