summaryrefslogtreecommitdiffstats
path: root/usr.sbin
diff options
context:
space:
mode:
authorngie <ngie@FreeBSD.org>2016-04-22 05:52:40 +0000
committerngie <ngie@FreeBSD.org>2016-04-22 05:52:40 +0000
commitf03b0d84d42dbd3c12ed29430273a59ff70b8a0a (patch)
treea45d2b38b4682250357e6c1e3cdd9e5a01bf6722 /usr.sbin
parent075957409e5fa19bcfde30525ea3a71880195bf7 (diff)
downloadFreeBSD-src-f03b0d84d42dbd3c12ed29430273a59ff70b8a0a.zip
FreeBSD-src-f03b0d84d42dbd3c12ed29430273a59ff70b8a0a.tar.gz
Don't use `entry` after free in the "already in lists" case
Return with 0 as it isn't an error. MFC after: 1 week CID: 1006085 Reported by: Coverity Obtained from: Isilon OneFS (part of r493633) Submitted by: Thor Steingrimsson <thor.steingrimsson@isilon.com> Sponsored by: EMC / Isilon Storage Division
Diffstat (limited to 'usr.sbin')
-rw-r--r--usr.sbin/bsnmpd/tools/libbsnmptools/bsnmpimport.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/usr.sbin/bsnmpd/tools/libbsnmptools/bsnmpimport.c b/usr.sbin/bsnmpd/tools/libbsnmptools/bsnmpimport.c
index b92532f..67a44d9 100644
--- a/usr.sbin/bsnmpd/tools/libbsnmptools/bsnmpimport.c
+++ b/usr.sbin/bsnmpd/tools/libbsnmptools/bsnmpimport.c
@@ -693,6 +693,7 @@ snmp_import_table(struct snmp_toolinfo *snmptoolctx, struct snmp_oid2str *obj)
/* Same entry already present in lists. */
free(entry->string);
free(entry);
+ return (0);
}
(void) snmp_import_update_table(ENTRY_INDEX, entry);
OpenPOWER on IntegriCloud