]> gitweb.michael.orlitzky.com - libsvgtiny.git/commitdiff
src/svgtiny_css.c: fix uninitalized use in node_is_lang()
authorMichael Orlitzky <michael@orlitzky.com>
Sun, 20 Oct 2024 22:17:52 +0000 (18:17 -0400)
committerMichael Orlitzky <michael@orlitzky.com>
Sun, 20 Oct 2024 22:17:52 +0000 (18:17 -0400)
If _node_has_attribute_substring() returns an error before it has a
chance to overwrite the "match" variable with true/false, then we may
try to access "match" while it is uninitialized. We may as well just
initialize it to false (no match) to avoid this corner case.

src/svgtiny_css.c

index 4856ac3f7bdefe4856f4bbf49415b6bf2c796415..d69fa5c827723dd3312674b6980c7cb3c1377184 100644 (file)
@@ -1796,7 +1796,10 @@ static css_error node_is_lang(void *pw, void *node,
        dom_exception d_err;
        dom_node *n; /* current node */
        dom_node *p; /* parent node */
-       bool match;  /* retval from node_has_attribute_substring() */
+
+       /* retval from node_has_attribute_substring(), must be
+        * initialized in case that function bails early */
+       bool match = false;
 
        /* Define the attribute name "lang" that we're looking for.
         * We only use a css_qname here because that's what the