projects
/
collectd.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Move the variable declaration closer to use
[collectd.git]
/
src
/
snmp.c
diff --git
a/src/snmp.c
b/src/snmp.c
index
8ff4009
..
98da2b8
100644
(file)
--- a/
src/snmp.c
+++ b/
src/snmp.c
@@
-99,7
+99,6
@@
struct host_definition_s {
c_complain_t complaint;
data_definition_t **data_list;
int data_list_len;
c_complain_t complaint;
data_definition_t **data_list;
int data_list_len;
-
int bulk_size;
};
typedef struct host_definition_s host_definition_t;
int bulk_size;
};
typedef struct host_definition_s host_definition_t;
@@
-822,7
+821,7
@@
static int csnmp_config_add_host(oconfig_item_t *ci) {
break;
}
if (hd->bulk_size > 0 && hd->version < 2) {
break;
}
if (hd->bulk_size > 0 && hd->version < 2) {
- WARNING("snmp plugin: Bulk transfer
ts is only available for snmp
v2 and "
+ WARNING("snmp plugin: Bulk transfer
s is only available for SNMP
v2 and "
"later, host '%s' is configured as version '%d'",
hd->name, hd->version);
}
"later, host '%s' is configured as version '%d'",
hd->name, hd->version);
}
@@
-1582,7
+1581,7
@@
static int csnmp_read_table(host_definition_t *host, data_definition_t *data) {
csnmp_oid_type_t oid_list_todo[oid_list_len];
int status;
csnmp_oid_type_t oid_list_todo[oid_list_len];
int status;
- size_t i
, j
;
+ size_t i;
/* `value_list_head' and `value_cells_tail' implement a linked list for each
* value. `instance_cells_head' and `instance_cells_tail' implement a linked
/* `value_list_head' and `value_cells_tail' implement a linked list for each
* value. `instance_cells_head' and `instance_cells_tail' implement a linked
@@
-1665,7
+1664,7
@@
static int csnmp_read_table(host_definition_t *host, data_definition_t *data) {
status = 0;
while (status == 0) {
status = 0;
while (status == 0) {
- /* If SNMP v2 and later and bulk transfer
t
enabled, use GETBULK PDU */
+ /* If SNMP v2 and later and bulk transfer
s
enabled, use GETBULK PDU */
if (host->version > 1 && host->bulk_size > 0) {
req = snmp_pdu_create(SNMP_MSG_GETBULK);
req->non_repeaters = 0;
if (host->version > 1 && host->bulk_size > 0) {
req = snmp_pdu_create(SNMP_MSG_GETBULK);
req->non_repeaters = 0;
@@
-1778,13
+1777,13
@@
static int csnmp_read_table(host_definition_t *host, data_definition_t *data) {
continue;
}
continue;
}
+ size_t j;
for (vb = res->variables, j = 0; (vb != NULL);
vb = vb->next_variable, j++) {
for (vb = res->variables, j = 0; (vb != NULL);
vb = vb->next_variable, j++) {
+ i = j;
/* If bulk request is active convert value index of the extra value */
if (host->version > 1 && host->bulk_size > 0) {
/* If bulk request is active convert value index of the extra value */
if (host->version > 1 && host->bulk_size > 0) {
- i = j % oid_list_todo_num;
- } else {
- i = j;
+ i %= oid_list_todo_num;
}
/* Calculate value index from todo list */
while ((i < oid_list_len) && !oid_list_todo[i]) {
}
/* Calculate value index from todo list */
while ((i < oid_list_len) && !oid_list_todo[i]) {