Lynne:
> +    dst->opaque    = av_buffer_ref(src->opaque);
> +    if (!dst->opaque) {
> +        ret = AVERROR(ENOMEM);
> +        goto fail;
> +    }

According to the doxy, opaque can be left unset (and will be given that
it is a new field), yet av_buffer_ref(NULL) will crash. We have
av_buffer_replace() for something like that.

How has this even been tested?

- Andreas

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to