ttmlparse: Fix potential NULL dereference identified by coverity
authorChris Bass <floobleflam@gmail.com>
Wed, 19 Apr 2017 13:07:06 +0000 (14:07 +0100)
committerSebastian Dröge <sebastian@centricular.com>
Wed, 19 Apr 2017 17:28:31 +0000 (18:28 +0100)
Hopefully fixes CID #1405131.

ext/ttml/ttmlparse.c

index 016523c..cb54fc3 100644 (file)
@@ -734,9 +734,9 @@ ttml_style_set_inherit (TtmlStyleSet * parent, TtmlStyleSet * child)
      * fontSize based on the parent's fontSize; otherwise, we simply keep
      * the value defined in the child's style set. */
     if (g_strcmp0 ((const gchar *) attr_name, "fontSize") == 0
-        && ttml_style_set_contains_attr (ret, (const gchar *) attr_name)) {
+        && ttml_style_set_contains_attr (ret, "fontSize")) {
       const gchar *original_child_font_size =
-          ttml_style_set_get_attr (child, "fontSize");
+          ttml_style_set_get_attr (ret, "fontSize");
       gchar *scaled_child_font_size =
           ttml_get_relative_font_size ((const gchar *) attr_value,
           original_child_font_size);