diff options
author | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2022-06-14 01:12:14 +0200 |
---|---|---|
committer | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2022-06-16 18:39:46 +0200 |
commit | 9ff0fbbc0ad095f4364b4d86ec7958908c6d737d (patch) | |
tree | 70e994dd75fae22ff3265d6a682423d1c1aee5f2 | |
parent | 6b5e3590c761aaf8e1ccceee8eadd8d49cf9c0af (diff) |
avformat/mxfdec: Offload allocating string to av_dict_set()
Reviewed-by: Tomas Härdin <tjoppen@acc.umu.se>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
-rw-r--r-- | libavformat/mxfdec.c | 22 |
1 files changed, 8 insertions, 14 deletions
diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 77bde7c3fe..400941c348 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -2118,16 +2118,12 @@ static int mxf_is_intra_only(MXFDescriptor *descriptor) &descriptor->essence_codec_ul)->id != AV_CODEC_ID_NONE; } -static int mxf_umid_to_str(UID ul, UID uid, char **str) +static void mxf_umid_to_str(const UID ul, const UID uid, + char str[2 + sizeof(UID) * 4 + 1]) { - char *p; - p = *str = av_mallocz(sizeof(UID) * 4 + 2 + 1); - if (!p) - return AVERROR(ENOMEM); - snprintf(p, 2 + 1, "0x"); - ff_data_to_hex(p + 2, ul, sizeof(UID), 0); - ff_data_to_hex(p + 2 + 2 * sizeof(UID), uid, sizeof(UID), 0); - return 0; + snprintf(str, 2 + sizeof(UID) * 4 + 1, "0x"); + ff_data_to_hex(str + 2, ul, sizeof(UID), 0); + ff_data_to_hex(str + 2 + 2 * sizeof(UID), uid, sizeof(UID), 0); } static int mxf_version_to_str(uint16_t major, uint16_t minor, uint16_t tertiary, @@ -2141,13 +2137,11 @@ static int mxf_version_to_str(uint16_t major, uint16_t minor, uint16_t tertiary, static int mxf_add_umid_metadata(AVDictionary **pm, const char *key, MXFPackage* package) { - char *str; - int ret; + char str[2 + 4 * sizeof(UID) + 1]; if (!package) return 0; - if ((ret = mxf_umid_to_str(package->package_ul, package->package_uid, &str)) < 0) - return ret; - av_dict_set(pm, key, str, AV_DICT_DONT_STRDUP_VAL); + mxf_umid_to_str(package->package_ul, package->package_uid, str); + av_dict_set(pm, key, str, 0); return 0; } |