Justin Ruggles <[email protected]> writes: > @@ -728,9 +729,11 @@ static int frame_thread_init(AVCodecContext *avctx) > > update_context_from_thread(avctx, copy, 1); > } else { > - copy->is_copy = 1; > copy->priv_data = av_malloc(codec->priv_data_size); > memcpy(copy->priv_data, src->priv_data, codec->priv_data_size); > + copy->internal = av_malloc(sizeof(AVCodecInternal)); > + *(copy->internal) = *(src->internal); > + copy->internal->is_copy = 1;
Missing check of av_malloc() return value. Yes, there's another missing one just above it. That is no excuse. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
