]> git.sesse.net Git - ffmpeg/commit
avutil/opt: Don't use NULL for %s string in a log message
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Sun, 29 Mar 2020 03:03:56 +0000 (05:03 +0200)
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Wed, 1 Apr 2020 02:09:18 +0000 (04:09 +0200)
commita500b975a87174f1994cf497892c2529cf42e834
treeb14bbff2e8c2d08d1649ef50b61c59b9b7ef8eb0
parentedee0d01048d26947053ca1657f11f65c21bd7bd
avutil/opt: Don't use NULL for %s string in a log message

If one calls av_opt_set() with an incorrect string to set the value of
an option of type AV_OPT_TYPE_VIDEO_RATE, the given string is used in a
log message via %s. This also happens when the string is actually a
nullpointer in which case using it for %s is forbidden.

This commit changes this by erroring out early in case of a nullpointer.

This also fixes a warning from GCC 9.2:
"ā€˜%sā€™ directive argument is null [-Wformat-overflow=]"

Reviewed-by: Anton Khirnov <anton@khirnov.net>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
libavutil/opt.c