[FFmpeg-devel] [PATCH 1/5] avformat/dashenc: fix invalid pointer access if avio_get_dyn_buf failed

Limin Wang lance.lmwang at gmail.com
Thu Apr 30 15:52:34 EEST 2020

On Thu, Apr 30, 2020 at 12:34:02PM +0200, Nicolas George wrote:
> lance.lmwang at gmail.com (12020-04-30):
> > Sorry, the old code will segment fault but the new code will not if error
> > happened. so I have no idea what's to compare the output as it's error.
> A segfault is better than corrupted data and better than lost data. So
> your task is to examine the output file and check whether it is
> corrupted and whether it contains all the data it should contain.
> > Also all of the code which are using avio_get_dyn_buf() didn't check the size
> > for error. I had to say the avio_get_dyn_buf() is designed very well.
> > 
> > Can we change avio_get_dyn_buf() to return the error directly? also the 
> > description of API isn't clear for the error condition.
> It seems avio_get_dyn_buf() is not a very good API, indeed. I do not
> know if it can be fixed. It is public, and therefore hard to change.
> I have been working on a better API for this kind of thing. Can I count
> on your support when I propose it again?

That's good to hear, please count me.

> Regards,
> -- 
>   Nicolas George

> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".

Limin Wang

More information about the ffmpeg-devel mailing list