summaryrefslogtreecommitdiff
path: root/libavfilter/vf_minterpolate.c
diff options
context:
space:
mode:
authorMarton Balint <cus@passwd.hu>2020-05-14 22:53:14 +0200
committerMarton Balint <cus@passwd.hu>2020-05-15 18:35:51 +0200
commit7f3a9462161aa5ad6cb498cc3a191960e4e77f51 (patch)
treeb8de22f8e610bd99235eef8f5da9b4ef25c60774 /libavfilter/vf_minterpolate.c
parent5913cd4e6c4b8b37c4933123212e90e6970ff19d (diff)
downloadffmpeg-7f3a9462161aa5ad6cb498cc3a191960e4e77f51.tar.gz
Revert "avfilter/vf_minterpolate: if metadata lavfi.scd.mafd exists, we'll use it first"
This reverts commit d88e1c9838dbcfe29d7835f2705ffc9ee6a36bf3. Fixes null pointer dereference. Signed-off-by: Marton Balint <cus@passwd.hu>
Diffstat (limited to 'libavfilter/vf_minterpolate.c')
-rw-r--r--libavfilter/vf_minterpolate.c16
1 files changed, 4 insertions, 12 deletions
diff --git a/libavfilter/vf_minterpolate.c b/libavfilter/vf_minterpolate.c
index ef47140946..c9ce80420d 100644
--- a/libavfilter/vf_minterpolate.c
+++ b/libavfilter/vf_minterpolate.c
@@ -834,19 +834,11 @@ static int detect_scene_change(MIContext *mi_ctx)
ptrdiff_t linesize2 = mi_ctx->frames[2].avf->linesize[0];
if (mi_ctx->scd_method == SCD_METHOD_FDIFF) {
- double ret = 0, mafd = HUGE_VAL, diff;
+ double ret = 0, mafd, diff;
uint64_t sad;
- AVDictionaryEntry *e_mafd = NULL;
- char *tail = NULL;
-
- e_mafd = av_dict_get(mi_ctx->frames[2].avf->metadata, "lavfi.scd.mafd", NULL, AV_DICT_MATCH_CASE);
- if (e_mafd)
- mafd = strtod(e_mafd->value, &tail);
- if (*tail || mafd == HUGE_VAL) {
- mi_ctx->sad(p1, linesize1, p2, linesize2, me_ctx->width, me_ctx->height, &sad);
- emms_c();
- mafd = (double) sad * 100.0 / (me_ctx->height * me_ctx->width) / (1 << mi_ctx->bitdepth);
- }
+ mi_ctx->sad(p1, linesize1, p2, linesize2, me_ctx->width, me_ctx->height, &sad);
+ emms_c();
+ mafd = (double) sad * 100.0 / (me_ctx->height * me_ctx->width) / (1 << mi_ctx->bitdepth);
diff = fabs(mafd - mi_ctx->prev_mafd);
ret = av_clipf(FFMIN(mafd, diff), 0, 100.0);
mi_ctx->prev_mafd = mafd;