From 557668e8cbd9695cc45a1135e158b81b5c2b6496 Mon Sep 17 00:00:00 2001 From: Andreas Rheinhardt Date: Sun, 10 Nov 2019 02:48:21 +0100 Subject: [PATCH] avformat/dss: Use AV_DICT_DONT_STRDUP_VAL to save a malloc+memcpy Reviewed-by: Paul B Mahol Signed-off-by: Andreas Rheinhardt --- libavformat/dss.c | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/libavformat/dss.c b/libavformat/dss.c index 8e0b72fbe5..ae7a259526 100644 --- a/libavformat/dss.c +++ b/libavformat/dss.c @@ -103,15 +103,11 @@ static int dss_read_metadata_string(AVFormatContext *s, unsigned int offset, ret = avio_read(s->pb, value, size); if (ret < size) { - ret = ret < 0 ? ret : AVERROR_EOF; - goto exit; + av_free(value); + return ret < 0 ? ret : AVERROR_EOF; } - ret = av_dict_set(&s->metadata, key, value, 0); - -exit: - av_free(value); - return ret; + return av_dict_set(&s->metadata, key, value, AV_DICT_DONT_STRDUP_VAL); } static int dss_read_header(AVFormatContext *s)