Merge branch 'collectd-4.10' into collectd-5.3
authorFlorian Forster <octo@collectd.org>
Mon, 18 Aug 2014 07:33:29 +0000 (09:33 +0200)
committerFlorian Forster <octo@collectd.org>
Mon, 18 Aug 2014 07:33:29 +0000 (09:33 +0200)
1  2 
configure.in
src/configfile.c
src/curl.c
src/curl_json.c
src/curl_xml.c
src/snmp.c
src/write_http.c

diff --cc configure.in
        then
                with_libxml2="yes"
        else
 -              with_libxml2="no (pkg-config doesn't know library)"
 +              with_libxml2="no (pkg-config doesn't know libxml-2.0)"
        fi
  
-       pkg-config --exists libvirt 2>/dev/null
+       $PKG_CONFIG --exists libvirt 2>/dev/null
        if test "$?" = "0"
        then
                with_libvirt="yes"
@@@ -572,25 -515,13 +578,26 @@@ static int cf_include_all (oconfig_item
                        continue;
                }
  
 -              new = cf_read_generic (old->values[0].value.string, depth + 1);
 +              for (j = 0; j < old->children_num; ++j)
 +              {
 +                      oconfig_item_t *child = old->children + j;
 +
 +                      if (strcasecmp (child->key, "Filter") == 0)
 +                              cf_util_get_string (child, &pattern);
 +                      else
 +                              ERROR ("configfile: Option `%s' not allowed in <Include> block.",
 +                                              child->key);
 +              }
 +
 +              new = cf_read_generic (old->values[0].value.string, pattern, depth + 1);
 +              sfree (pattern);
 +
                if (new == NULL)
 -                      continue;
 +                      return (-1);
  
                /* Now replace the i'th child in `root' with `new'. */
-               cf_ci_replace_child (root, new, i);
+               if (cf_ci_replace_child (root, new, i) < 0)
+                       return (-1);
  
                /* ... and go back to the new i'th child. */
                --i;
diff --cc src/curl.c
Simple merge
diff --cc src/curl_json.c
Simple merge
diff --cc src/curl_xml.c
@@@ -383,9 -342,9 +383,9 @@@ static int cx_handle_instance_xpath (xm
    memset (vl->type_instance, 0, sizeof (vl->type_instance));
  
    /* If the base xpath returns more than one block, the result is assumed to be
 -   * a table. The `Instnce' option is not optional in this case. Check for the
 +   * a table. The `Instance' option is not optional in this case. Check for the
     * condition and inform the user. */
-   if (is_table && (vl->type_instance == NULL))
+   if (is_table)
    {
      WARNING ("curl_xml plugin: "
          "Base-XPath %s is a table (more than one result was returned), "
diff --cc src/snmp.c
Simple merge
Simple merge