svn commit: r313222 - stable/10/contrib/bsnmp/snmp_usm
Ngie Cooper
ngie at FreeBSD.org
Sat Feb 4 16:37:45 UTC 2017
Author: ngie
Date: Sat Feb 4 16:37:43 2017
New Revision: 313222
URL: https://svnweb.freebsd.org/changeset/base/313222
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/10/contrib/bsnmp/snmp_usm/usm_snmp.c
Directory Properties:
stable/10/ (props changed)
Modified: stable/10/contrib/bsnmp/snmp_usm/usm_snmp.c
==============================================================================
--- stable/10/contrib/bsnmp/snmp_usm/usm_snmp.c Sat Feb 4 16:36:07 2017 (r313221)
+++ stable/10/contrib/bsnmp/snmp_usm/usm_snmp.c Sat Feb 4 16:37:43 2017 (r313222)
@@ -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