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".