On 07/11/2016 02:23 AM, Marton Balint wrote:

+    ret = av_dict_copy(&dst->metadata, src->metadata, 0);
+    if (ret < 0)
+        return ret;

Since you are resetting every existing field in dst, it might make sense to free the metadata dictionary as well, before copying items to it.

Thanks for noticing, I've fixed that, I will wait for Nicolas's reply regarding duration field and then send the modified patch.

Regards,
Jan

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

Reply via email to