|
 |
9a6c41 |
983116: net-snmp query fails after update to 1:5.5-44.el6_4.2
|
|
|
9a6c41 |
|
|
|
9a6c41 |
|
|
|
9a6c41 |
commit e41c2f574c25d8dd273f2406eeeac19bc2ae16db
|
|
|
9a6c41 |
Author: Jan Safranek <jsafranek@users.sourceforge.net>
|
|
|
9a6c41 |
Date: Mon Aug 12 14:30:51 2013 +0200
|
|
|
9a6c41 |
|
|
|
9a6c41 |
CHANGES: snmplib: Fixed parsing of sequences.
|
|
|
9a6c41 |
|
|
|
9a6c41 |
Don't overwrite 'data' variable, it's used when parsing bulk responses.
|
|
|
9a6c41 |
|
|
|
9a6c41 |
diff --git a/snmplib/snmp_api.c b/snmplib/snmp_api.c
|
|
|
9a6c41 |
index 403ea2b..e67945f 100644
|
|
|
9a6c41 |
--- a/snmplib/snmp_api.c
|
|
|
9a6c41 |
+++ b/snmplib/snmp_api.c
|
|
|
9a6c41 |
@@ -4537,6 +4537,7 @@ snmp_pdu_parse(netsnmp_pdu *pdu, u_char * data, size_t * length)
|
|
|
9a6c41 |
size_t four;
|
|
|
9a6c41 |
netsnmp_variable_list *vp = NULL;
|
|
|
9a6c41 |
oid objid[MAX_OID_LEN];
|
|
|
9a6c41 |
+ u_char *p;
|
|
|
9a6c41 |
|
|
|
9a6c41 |
/*
|
|
|
9a6c41 |
* Get the PDU type
|
|
|
9a6c41 |
@@ -4709,10 +4710,10 @@ snmp_pdu_parse(netsnmp_pdu *pdu, u_char * data, size_t * length)
|
|
|
9a6c41 |
case ASN_INTEGER:
|
|
|
9a6c41 |
vp->val.integer = (long *) vp->buf;
|
|
|
9a6c41 |
vp->val_len = sizeof(long);
|
|
|
9a6c41 |
- data = asn_parse_int(var_val, &len, &vp->type,
|
|
|
9a6c41 |
+ p = asn_parse_int(var_val, &len, &vp->type,
|
|
|
9a6c41 |
(long *) vp->val.integer,
|
|
|
9a6c41 |
sizeof(*vp->val.integer));
|
|
|
9a6c41 |
- if (!data)
|
|
|
9a6c41 |
+ if (!p)
|
|
|
9a6c41 |
return -1;
|
|
|
9a6c41 |
break;
|
|
|
9a6c41 |
case ASN_COUNTER:
|
|
|
9a6c41 |
@@ -4721,10 +4722,10 @@ snmp_pdu_parse(netsnmp_pdu *pdu, u_char * data, size_t * length)
|
|
|
9a6c41 |
case ASN_UINTEGER:
|
|
|
9a6c41 |
vp->val.integer = (long *) vp->buf;
|
|
|
9a6c41 |
vp->val_len = sizeof(u_long);
|
|
|
9a6c41 |
- data = asn_parse_unsigned_int(var_val, &len, &vp->type,
|
|
|
9a6c41 |
+ p = asn_parse_unsigned_int(var_val, &len, &vp->type,
|
|
|
9a6c41 |
(u_long *) vp->val.integer,
|
|
|
9a6c41 |
vp->val_len);
|
|
|
9a6c41 |
- if (!data)
|
|
|
9a6c41 |
+ if (!p)
|
|
|
9a6c41 |
return -1;
|
|
|
9a6c41 |
break;
|
|
|
9a6c41 |
#ifdef NETSNMP_WITH_OPAQUE_SPECIAL_TYPES
|
|
|
9a6c41 |
@@ -4734,37 +4735,37 @@ snmp_pdu_parse(netsnmp_pdu *pdu, u_char * data, size_t * length)
|
|
|
9a6c41 |
case ASN_COUNTER64:
|
|
|
9a6c41 |
vp->val.counter64 = (struct counter64 *) vp->buf;
|
|
|
9a6c41 |
vp->val_len = sizeof(struct counter64);
|
|
|
9a6c41 |
- data = asn_parse_unsigned_int64(var_val, &len, &vp->type,
|
|
|
9a6c41 |
+ p = asn_parse_unsigned_int64(var_val, &len, &vp->type,
|
|
|
9a6c41 |
(struct counter64 *) vp->val.
|
|
|
9a6c41 |
counter64, vp->val_len);
|
|
|
9a6c41 |
- if (!data)
|
|
|
9a6c41 |
+ if (!p)
|
|
|
9a6c41 |
return -1;
|
|
|
9a6c41 |
break;
|
|
|
9a6c41 |
#ifdef NETSNMP_WITH_OPAQUE_SPECIAL_TYPES
|
|
|
9a6c41 |
case ASN_OPAQUE_FLOAT:
|
|
|
9a6c41 |
vp->val.floatVal = (float *) vp->buf;
|
|
|
9a6c41 |
vp->val_len = sizeof(float);
|
|
|
9a6c41 |
- data = asn_parse_float(var_val, &len, &vp->type,
|
|
|
9a6c41 |
+ p = asn_parse_float(var_val, &len, &vp->type,
|
|
|
9a6c41 |
vp->val.floatVal, vp->val_len);
|
|
|
9a6c41 |
- if (!data)
|
|
|
9a6c41 |
+ if (!p)
|
|
|
9a6c41 |
return -1;
|
|
|
9a6c41 |
break;
|
|
|
9a6c41 |
case ASN_OPAQUE_DOUBLE:
|
|
|
9a6c41 |
vp->val.doubleVal = (double *) vp->buf;
|
|
|
9a6c41 |
vp->val_len = sizeof(double);
|
|
|
9a6c41 |
- data = asn_parse_double(var_val, &len, &vp->type,
|
|
|
9a6c41 |
+ p = asn_parse_double(var_val, &len, &vp->type,
|
|
|
9a6c41 |
vp->val.doubleVal, vp->val_len);
|
|
|
9a6c41 |
- if (!data)
|
|
|
9a6c41 |
+ if (!p)
|
|
|
9a6c41 |
return -1;
|
|
|
9a6c41 |
break;
|
|
|
9a6c41 |
case ASN_OPAQUE_I64:
|
|
|
9a6c41 |
vp->val.counter64 = (struct counter64 *) vp->buf;
|
|
|
9a6c41 |
vp->val_len = sizeof(struct counter64);
|
|
|
9a6c41 |
- data = asn_parse_signed_int64(var_val, &len, &vp->type,
|
|
|
9a6c41 |
+ p = asn_parse_signed_int64(var_val, &len, &vp->type,
|
|
|
9a6c41 |
(struct counter64 *) vp->val.counter64,
|
|
|
9a6c41 |
sizeof(*vp->val.counter64));
|
|
|
9a6c41 |
|
|
|
9a6c41 |
- if (!data)
|
|
|
9a6c41 |
+ if (!p)
|
|
|
9a6c41 |
return -1;
|
|
|
9a6c41 |
break;
|
|
|
9a6c41 |
#endif /* NETSNMP_WITH_OPAQUE_SPECIAL_TYPES */
|
|
|
9a6c41 |
@@ -4783,15 +4784,15 @@ snmp_pdu_parse(netsnmp_pdu *pdu, u_char * data, size_t * length)
|
|
|
9a6c41 |
if (vp->val.string == NULL) {
|
|
|
9a6c41 |
return -1;
|
|
|
9a6c41 |
}
|
|
|
9a6c41 |
- data = asn_parse_string(var_val, &len, &vp->type, vp->val.string,
|
|
|
9a6c41 |
+ p = asn_parse_string(var_val, &len, &vp->type, vp->val.string,
|
|
|
9a6c41 |
&vp->val_len);
|
|
|
9a6c41 |
- if (!data)
|
|
|
9a6c41 |
+ if (!p)
|
|
|
9a6c41 |
return -1;
|
|
|
9a6c41 |
break;
|
|
|
9a6c41 |
case ASN_OBJECT_ID:
|
|
|
9a6c41 |
vp->val_len = MAX_OID_LEN;
|
|
|
9a6c41 |
- data = asn_parse_objid(var_val, &len, &vp->type, objid, &vp->val_len);
|
|
|
9a6c41 |
- if (!data)
|
|
|
9a6c41 |
+ p = asn_parse_objid(var_val, &len, &vp->type, objid, &vp->val_len);
|
|
|
9a6c41 |
+ if (!p)
|
|
|
9a6c41 |
return -1;
|
|
|
9a6c41 |
vp->val_len *= sizeof(oid);
|
|
|
9a6c41 |
vp->val.objid = (oid *) malloc(vp->val_len);
|
|
|
9a6c41 |
@@ -4810,9 +4811,9 @@ snmp_pdu_parse(netsnmp_pdu *pdu, u_char * data, size_t * length)
|
|
|
9a6c41 |
if (vp->val.bitstring == NULL) {
|
|
|
9a6c41 |
return -1;
|
|
|
9a6c41 |
}
|
|
|
9a6c41 |
- data = asn_parse_bitstring(var_val, &len, &vp->type,
|
|
|
9a6c41 |
+ p = asn_parse_bitstring(var_val, &len, &vp->type,
|
|
|
9a6c41 |
vp->val.bitstring, &vp->val_len);
|
|
|
9a6c41 |
- if (!data)
|
|
|
9a6c41 |
+ if (!p)
|
|
|
9a6c41 |
return -1;
|
|
|
9a6c41 |
break;
|
|
|
9a6c41 |
default:
|