projects
/
collectd.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
ChangeLog: Updated for version 4.10.0.
[collectd.git]
/
src
/
netlink.c
diff --git
a/src/netlink.c
b/src/netlink.c
index
d7d1a26
..
49c4e99
100644
(file)
--- a/
src/netlink.c
+++ b/
src/netlink.c
@@
-204,8
+204,8
@@
static void submit_two (const char *dev, const char *type,
plugin_dispatch_values (&vl);
} /* void submit_two */
plugin_dispatch_values (&vl);
} /* void submit_two */
-static int link_filter (const struct sockaddr_nl *sa,
- 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,7
+314,7
@@
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,
+static int qos_filter (const struct sockaddr_nl
__attribute__((unused))
*sa,
struct nlmsghdr *nmh, void *args)
{
struct tcmsg *msg;
struct nlmsghdr *nmh, void *args)
{
struct tcmsg *msg;
@@
-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",
@@
-525,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;
@@
-580,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;