diff options
author | Stephen Hutchinson <qyot27@gmail.com> | 2022-02-16 19:37:18 -0500 |
---|---|---|
committer | Stephen Hutchinson <qyot27@gmail.com> | 2022-02-18 17:11:19 -0500 |
commit | 0d34e21282b0a0f166cf97046fd35587afdc0515 (patch) | |
tree | 3b4866aa4a0708b004ed28295c884d92cee9eb77 /libavformat | |
parent | 3abb80f65b44a803dda498e111b8c96a80f77d10 (diff) | |
download | ffmpeg-0d34e21282b0a0f166cf97046fd35587afdc0515.tar.gz |
avformat/avisynth: fix frameprop version check
Trying to be clever about determining between interface version 8
and 8.1 ended up with pre-8.1 versions of AviSynth+ segfaulting.
The amount of time between interface version 8.1 and 9 is small,
so just restrict the frameprop awareness to version 9 and call it
a day.
Signed-off-by: Stephen Hutchinson <qyot27@gmail.com>
Diffstat (limited to 'libavformat')
-rw-r--r-- | libavformat/avisynth.c | 25 |
1 files changed, 6 insertions, 19 deletions
diff --git a/libavformat/avisynth.c b/libavformat/avisynth.c index 8bc39869a3..2bd0c6949b 100644 --- a/libavformat/avisynth.c +++ b/libavformat/avisynth.c @@ -19,8 +19,6 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#include <stdbool.h> - #include "libavutil/attributes.h" #include "libavutil/internal.h" @@ -502,24 +500,13 @@ static int avisynth_create_stream_video(AVFormatContext *s, AVStream *st) /* Read AviSynth+'s frame properties to set additional info. * * Due to a bug preventing the C interface from accessing frame - * properties in earlier versions of interface version 8, only - * enable this if we detect version 8.1 at the minimum. */ - - if (!avs_library.avs_get_env_property) { - av_log(s, AV_LOG_TRACE, "%s\n", - "avs_get_env_property does not exist in AviSynth library; frame properties won't be checked."); - frameprop = false; - } else { - if (avs_library.avs_get_env_property(avs->env, AVS_AEP_INTERFACE_BUGFIX)) { - av_log(s, AV_LOG_TRACE, "%s\n", "Using interface version 8.1 or higher, reading frame properties."); - frameprop = true; - } else { - av_log(s, AV_LOG_TRACE, "%s\n", "Using interface version 8.0, need 8.1+ to read frame properties."); - frameprop = false; - } - } + * properties in earlier versions of interface version 8, and + * previous attempts at being clever resulting in pre-8 versions + * of AviSynth+ segfaulting, only enable this if we detect + * version 9 at the minimum. Technically, 8.1 works, but the time + * distance between 8.1 and 9 is very small, so just restrict it to 9. */ - if (frameprop = true) { + if (avs_library.avs_get_version(avs->clip) >= 9) { frame = avs_library.avs_get_frame(avs->clip, framedata); avsmap = avs_library.avs_get_frame_props_ro(avs->env, frame); |