[FFmpeg-devel] [PATCH 3/4] avutil/frame: Reimplement av_frame_new_side_data() without size=0 special case

Carl Eugen Hoyos ceffmpeg at gmail.com
Fri Feb 24 03:24:01 EET 2017


2017-02-24 1:19 GMT+01:00 Michael Niedermayer <michael at niedermayer.cc>:

> The problem is that functions like memcpy dont allow NULL even with
> a size of 0
>
> i can add checks for every memcpy() and any other function that ubsan
> finds if thats preferred or add a av_memcpy() that does allow NULL

No, it's preferable to fix the original issue, not adding random
work-arounds all over the codebase.

Thank you, Carl Eugen


More information about the ffmpeg-devel mailing list