projects
/
collectd.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Merge branch 'collectd-4.10' into collectd-5.0
[collectd.git]
/
src
/
netlink.c
diff --git
a/src/netlink.c
b/src/netlink.c
index
d6288e9
..
ef851d3
100644
(file)
--- a/
src/netlink.c
+++ b/
src/netlink.c
@@
-1,6
+1,6
@@
/**
* collectd - src/netlink.c
/**
* collectd - src/netlink.c
- * Copyright (C) 2007 Florian octo Forster
+ * Copyright (C) 2007
-2010
Florian octo Forster
*
* This program is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License as published by the
*
* This program is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License as published by the
@@
-16,7
+16,7
@@
* 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*
* Authors:
* 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*
* Authors:
- * Florian octo Forster <octo at
verplant
.org>
+ * Florian octo Forster <octo at
collectd
.org>
**/
#include "collectd.h"
**/
#include "collectd.h"
@@
-161,51
+161,51
@@
static int check_ignorelist (const char *dev,
} /* int check_ignorelist */
static void submit_one (const char *dev, const char *type,
} /* int check_ignorelist */
static void submit_one (const char *dev, const char *type,
- const char *type_instance,
counter
_t value)
+ const char *type_instance,
derive
_t value)
{
value_t values[1];
value_list_t vl = VALUE_LIST_INIT;
{
value_t values[1];
value_list_t vl = VALUE_LIST_INIT;
- values[0].
counter
= value;
+ values[0].
derive
= value;
vl.values = values;
vl.values_len = 1;
vl.values = values;
vl.values_len = 1;
- vl.time = time (NULL);
sstrncpy (vl.host, hostname_g, sizeof (vl.host));
sstrncpy (vl.plugin, "netlink", sizeof (vl.plugin));
sstrncpy (vl.host, hostname_g, sizeof (vl.host));
sstrncpy (vl.plugin, "netlink", sizeof (vl.plugin));
- strncpy (vl.plugin_instance, dev, sizeof (vl.plugin_instance));
+ sstrncpy (vl.plugin_instance, dev, sizeof (vl.plugin_instance));
+ sstrncpy (vl.type, type, sizeof (vl.type));
if (type_instance != NULL)
if (type_instance != NULL)
- strncpy (vl.type_instance, type_instance, sizeof (vl.type_instance));
+ s
s
trncpy (vl.type_instance, type_instance, sizeof (vl.type_instance));
- plugin_dispatch_values (
type,
&vl);
+ plugin_dispatch_values (&vl);
} /* void submit_one */
static void submit_two (const char *dev, const char *type,
const char *type_instance,
} /* void submit_one */
static void submit_two (const char *dev, const char *type,
const char *type_instance,
-
counter_t rx, counter
_t tx)
+
derive_t rx, derive
_t tx)
{
value_t values[2];
value_list_t vl = VALUE_LIST_INIT;
{
value_t values[2];
value_list_t vl = VALUE_LIST_INIT;
- values[0].
counter
= rx;
- values[1].
counter
= tx;
+ values[0].
derive
= rx;
+ values[1].
derive
= tx;
vl.values = values;
vl.values_len = 2;
vl.values = values;
vl.values_len = 2;
- vl.time = time (NULL);
sstrncpy (vl.host, hostname_g, sizeof (vl.host));
sstrncpy (vl.plugin, "netlink", sizeof (vl.plugin));
sstrncpy (vl.host, hostname_g, sizeof (vl.host));
sstrncpy (vl.plugin, "netlink", sizeof (vl.plugin));
- strncpy (vl.plugin_instance, dev, sizeof (vl.plugin_instance));
+ sstrncpy (vl.plugin_instance, dev, sizeof (vl.plugin_instance));
+ sstrncpy (vl.type, type, sizeof (vl.type));
if (type_instance != NULL)
if (type_instance != NULL)
- strncpy (vl.type_instance, type_instance, sizeof (vl.type_instance));
+ s
s
trncpy (vl.type_instance, type_instance, sizeof (vl.type_instance));
- plugin_dispatch_values (
type,
&vl);
+ plugin_dispatch_values (&vl);
} /* void submit_two */
} /* void submit_two */
-static int link_filter (const struct sockaddr_nl *sa,
-
const struct nlmsghdr *nmh, void
*args)
+static int link_filter (const struct sockaddr_nl
__attribute__((unused))
*sa,
+
struct nlmsghdr *nmh, void __attribute__((unused))
*args)
{
struct ifinfomsg *msg;
int msg_len;
{
struct ifinfomsg *msg;
int msg_len;
@@
-246,7
+246,7
@@
static int link_filter (const struct sockaddr_nl *sa,
/* Update the `iflist'. It's used to know which interfaces exist and query
* them later for qdiscs and classes. */
/* Update the `iflist'. It's used to know which interfaces exist and query
* them later for qdiscs and classes. */
- if (
msg->ifi_index >= iflist_len
)
+ if (
(msg->ifi_index >= 0) && ((size_t) msg->ifi_index >= iflist_len)
)
{
char **temp;
{
char **temp;
@@
-314,8
+314,8
@@
static int link_filter (const struct sockaddr_nl *sa,
return (0);
} /* int link_filter */
return (0);
} /* int link_filter */
-static int qos_filter (const struct sockaddr_nl *sa,
-
const
struct nlmsghdr *nmh, void *args)
+static int qos_filter (const struct sockaddr_nl
__attribute__((unused))
*sa,
+ struct nlmsghdr *nmh, void *args)
{
struct tcmsg *msg;
int msg_len;
{
struct tcmsg *msg;
int msg_len;
@@
-359,7
+359,8
@@
static int qos_filter (const struct sockaddr_nl *sa,
return (0);
}
return (0);
}
- if (msg->tcm_ifindex >= iflist_len)
+ if ((msg->tcm_ifindex >= 0)
+ && ((size_t) msg->tcm_ifindex >= iflist_len))
{
ERROR ("netlink plugin: qos_filter: msg->tcm_ifindex = %i "
">= iflist_len = %zu",
{
ERROR ("netlink plugin: qos_filter: msg->tcm_ifindex = %i "
">= iflist_len = %zu",
@@
-395,11
+396,10
@@
static int qos_filter (const struct sockaddr_nl *sa,
if (strcmp (tc_type, "filter") == 0)
numberic_id = msg->tcm_parent;
if (strcmp (tc_type, "filter") == 0)
numberic_id = msg->tcm_parent;
- snprintf (tc_inst, sizeof (tc_inst), "%s-%x:%x",
+ s
s
nprintf (tc_inst, sizeof (tc_inst), "%s-%x:%x",
(const char *) RTA_DATA (attrs[TCA_KIND]),
numberic_id >> 16,
numberic_id & 0x0000FFFF);
(const char *) RTA_DATA (attrs[TCA_KIND]),
numberic_id >> 16,
numberic_id & 0x0000FFFF);
- tc_inst[sizeof (tc_inst) - 1] = '\0';
}
DEBUG ("netlink plugin: qos_filter: got %s for %s (%i).",
}
DEBUG ("netlink plugin: qos_filter: got %s for %s (%i).",
@@
-421,9
+421,8
@@
static int qos_filter (const struct sockaddr_nl *sa,
struct gnet_stats_basic bs;
char type_instance[DATA_MAX_NAME_LEN];
struct gnet_stats_basic bs;
char type_instance[DATA_MAX_NAME_LEN];
- snprintf (type_instance, sizeof (type_instance), "%s-%s",
+ s
s
nprintf (type_instance, sizeof (type_instance), "%s-%s",
tc_type, tc_inst);
tc_type, tc_inst);
- type_instance[sizeof (type_instance) - 1] = '\0';
memset (&bs, '\0', sizeof (bs));
memcpy (&bs, RTA_DATA (attrs_stats[TCA_STATS_BASIC]),
memset (&bs, '\0', sizeof (bs));
memcpy (&bs, RTA_DATA (attrs_stats[TCA_STATS_BASIC]),
@@
-443,9
+442,8
@@
static int qos_filter (const struct sockaddr_nl *sa,
struct tc_stats ts;
char type_instance[DATA_MAX_NAME_LEN];
struct tc_stats ts;
char type_instance[DATA_MAX_NAME_LEN];
- snprintf (type_instance, sizeof (type_instance), "%s-%s",
+ s
s
nprintf (type_instance, sizeof (type_instance), "%s-%s",
tc_type, tc_inst);
tc_type, tc_inst);
- type_instance[sizeof (type_instance) - 1] = '\0';
memset(&ts, '\0', sizeof (ts));
memcpy(&ts, RTA_DATA (attrs[TCA_STATS]),
memset(&ts, '\0', sizeof (ts));
memcpy(&ts, RTA_DATA (attrs[TCA_STATS]),
@@
-528,9
+526,7
@@
static int ir_config (const char *key, const char *value)
}
else
{
}
else
{
- if ((strcasecmp (fields[0], "yes") == 0)
- || (strcasecmp (fields[0], "true") == 0)
- || (strcasecmp (fields[0], "on") == 0))
+ if (IS_TRUE (fields[0]))
ir_ignorelist_invert = 0;
else
ir_ignorelist_invert = 1;
ir_ignorelist_invert = 0;
else
ir_ignorelist_invert = 1;
@@
-583,9
+579,9
@@
static int ir_read (void)
/* `link_filter' will update `iflist' which is used here to iterate over all
* interfaces. */
/* `link_filter' will update `iflist' which is used here to iterate over all
* interfaces. */
- for (ifindex = 0; ifindex < iflist_len; ifindex++)
+ for (ifindex = 0;
(size_t)
ifindex < iflist_len; ifindex++)
{
{
-
in
t type_index;
+
size_
t type_index;
if (iflist[ifindex] == NULL)
continue;
if (iflist[ifindex] == NULL)
continue;