diff options
| author | cem <cem@FreeBSD.org> | 2016-05-11 16:54:34 +0000 |
|---|---|---|
| committer | cem <cem@FreeBSD.org> | 2016-05-11 16:54:34 +0000 |
| commit | 61b729e84bcc908317ca877bc1377b6e43e724fe (patch) | |
| tree | cb97f5f49b32ad652d207904779919a82b786981 /contrib/bsnmp/snmp_usm | |
| parent | a49d3e4c437979578949ff7ef69cb7b8690bad25 (diff) | |
| download | FreeBSD-src-61b729e84bcc908317ca877bc1377b6e43e724fe.zip FreeBSD-src-61b729e84bcc908317ca877bc1377b6e43e724fe.tar.gz | |
bsnmp: Don't overrun privkey buffer by copying wrong size
The 'priv_key' array is SNMP_PRIV_KEY_SIZ bytes, not SNMP_AUTH_KEY_SIZ.
Reported by: Coverity
CIDs: 1008326, 1009675
Sponsored by: EMC / Isilon Storage Division
Diffstat (limited to 'contrib/bsnmp/snmp_usm')
| -rw-r--r-- | contrib/bsnmp/snmp_usm/usm_snmp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/contrib/bsnmp/snmp_usm/usm_snmp.c b/contrib/bsnmp/snmp_usm/usm_snmp.c index d5350bc..d43baa9 100644 --- a/contrib/bsnmp/snmp_usm/usm_snmp.c +++ b/contrib/bsnmp/snmp_usm/usm_snmp.c @@ -360,7 +360,7 @@ op_usm_users(struct snmp_context *ctx, struct snmp_value *val, case LEAF_usmUserPrivKeyChange: case LEAF_usmUserOwnPrivKeyChange: memcpy(uuser->suser.priv_key, ctx->scratch->ptr1, - SNMP_AUTH_KEY_SIZ); + SNMP_PRIV_KEY_SIZ); free(ctx->scratch->ptr1); break; case LEAF_usmUserPublic: |
