From: Pierre-Anthony Lemieux <p...@palemieux.com>
Fixes Coverity issue #1512406
---
libavformat/imfdec.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/libavformat/imfdec.c b/libavformat/imfdec.c
index 5bbe7a53f8..fde91a6419 100644
--- a/libavformat/imfdec.c
+++ b/libavformat/imfdec.c
@@ -233,7 +233,12 @@ static int
parse_imf_asset_map_from_xml_dom(AVFormatContext *s,
asset = &(asset_map->assets[asset_map->asset_count]);
- if
(ff_imf_xml_read_uuid(ff_imf_xml_get_child_element_by_name(asset_element,
"Id"), asset->uuid)) {
+ if (!(node = ff_imf_xml_get_child_element_by_name(asset_element,
"Id"))) {
+ av_log(s, AV_LOG_ERROR, "Unable to parse asset map XML - missing
Id node\n");
+ return AVERROR_INVALIDDATA;
+ }
+
+ if (ff_imf_xml_read_uuid(node, asset->uuid)) {
av_log(s, AV_LOG_ERROR, "Could not parse UUID from asset in asset
map.\n");
return AVERROR_INVALIDDATA;
}
--
2.25.1
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
