diff mbox

[FFmpeg-devel] avformat/dashdec: fix pointer being freed was not allocated

Message ID 20190913163012.91715-1-hello.vectronic@gmail.com
State New
Headers show

Commit Message

vectronic Sept. 13, 2019, 4:30 p.m. UTC 
prevent attempt to call xmlFree if val was not allocated

fixes: 8135
Signed-off-by: vectronic <hello.vectronic@gmail.com>
---
 libavformat/dashdec.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
diff mbox

Patch

diff --git a/libavformat/dashdec.c b/libavformat/dashdec.c
index 4f725ba09a..8022ba9afe 100644
--- a/libavformat/dashdec.c
+++ b/libavformat/dashdec.c
@@ -1185,6 +1185,7 @@  static int parse_programinformation(AVFormatContext *s, xmlNodePtr node)
 
     node = xmlFirstElementChild(node);
     while (node) {
+        val = NULL;
         if (!av_strcasecmp(node->name, "Title")) {
             val = xmlNodeGetContent(node);
             if (val) {
@@ -1202,7 +1203,9 @@  static int parse_programinformation(AVFormatContext *s, xmlNodePtr node)
             }
         }
         node = xmlNextElementSibling(node);
-        xmlFree(val);
+        if (val) {
+            xmlFree(val);
+        }
     }
     return 0;
 }