Bind plugin: plug a few leaks

Fixes: #2303

diff --git a/src/bind.c b/src/bind.c
index 4860f1b..853b9c2 100644 (file)
--- a/src/bind.c
+++ b/src/bind.c
@@ -526,8 +526,10 @@ static int bind_parse_generic_name_value(const char *xpath_expression, /* {{{ */
         status = bind_xml_read_gauge(doc, counter, &value.gauge);
       else
         status = bind_xml_read_derive(doc, counter, &value.derive);
         status = bind_xml_read_gauge(doc, counter, &value.gauge);
       else
         status = bind_xml_read_derive(doc, counter, &value.derive);

-      if (status != 0)
+      if (status != 0) {
+        xmlFree(name);

         continue;
         continue;

+      }

 
       status = (*list_callback)(name, value, current_time, user_data);
       if (status == 0)
 
       status = (*list_callback)(name, value, current_time, user_data);
       if (status == 0)


@@ -659,12 +661,16 @@ static int bind_parse_generic_name_attr_value_list(
         status = bind_xml_read_gauge(doc, child, &value.gauge);
       else
         status = bind_xml_read_derive(doc, child, &value.derive);
         status = bind_xml_read_gauge(doc, child, &value.gauge);
       else
         status = bind_xml_read_derive(doc, child, &value.derive);

-      if (status != 0)
+      if (status != 0) {
+        xmlFree(attr_name);

         continue;
         continue;

+      }

 
       status = (*list_callback)(attr_name, value, current_time, user_data);
       if (status == 0)
         num_entries++;
 
       status = (*list_callback)(attr_name, value, current_time, user_data);
       if (status == 0)
         num_entries++;

+
+      xmlFree(attr_name);

     }
   }
 
     }
   }