svn commit: r313241 - stable/11/contrib/bsnmp/snmp_usm
Ngie Cooper
ngie at FreeBSD.org
Sat Feb 4 17:18:51 UTC 2017
Author: ngie
Date: Sat Feb 4 17:18:49 2017
New Revision: 313241
URL: https://svnweb.freebsd.org/changeset/base/313241
Log:
MFC r311394:
op_usm_users: don't deref uusers if it's NULL when SETting the value
Add an XXX comment to note that the conditional seems suspect given
how it's handled elsewhere in the SNMP_OP_SET case.
CID: 1008573
Modified:
stable/11/contrib/bsnmp/snmp_usm/usm_snmp.c
Directory Properties:
stable/11/ (props changed)
Modified: stable/11/contrib/bsnmp/snmp_usm/usm_snmp.c
==============================================================================
--- stable/11/contrib/bsnmp/snmp_usm/usm_snmp.c Sat Feb 4 17:17:38 2017 (r313240)
+++ stable/11/contrib/bsnmp/snmp_usm/usm_snmp.c Sat Feb 4 17:18:49 2017 (r313241)
@@ -169,8 +169,12 @@ op_usm_users(struct snmp_context *ctx, s
val->var.subs[sub - 1] != LEAF_usmUserCloneFrom)
return (SNMP_ERR_NOSUCHNAME);
+ /*
+ * XXX (ngie): need to investigate the MIB to determine how
+ * this is possible given some of the transitions below.
+ */
if (community != COMM_INITIALIZE &&
- uuser->type == StorageType_readOnly)
+ uuser != NULL && uuser->type == StorageType_readOnly)
return (SNMP_ERR_NOT_WRITEABLE);
switch (val->var.subs[sub - 1]) {
More information about the svn-src-stable
mailing list