On 04/05/2014 01:11 PM, Nidhi Makhijani wrote:
> @@ -400,6 +407,10 @@ static int decode_stream_header(NUTContext *nut)
> if (st->codec->extradata_size) {
> st->codec->extradata = av_mallocz(st->codec->extradata_size +
> FF_INPUT_BUFFER_PADDING_SIZE);
> + if (!st->codec->extradata) {
> + av_freep(st);
> + return AVERROR(ENOMEM);
> + }
'st' shouldn't be freed here. It is not allocated in this function.
As for the rest of the patch, av_freep() takes a pointer to a pointer.
-Justin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
