svn commit: r315206 - head/contrib/bsnmp/snmpd
Ngie Cooper
ngie at FreeBSD.org
Mon Mar 13 18:01:03 UTC 2017
Author: ngie
Date: Mon Mar 13 18:01:01 2017
New Revision: 315206
URL: https://svnweb.freebsd.org/changeset/base/315206
Log:
bsnmpd: fix segfault when trans_insert_port(..) is called with multiple
out of order addresses
Move `port->transport` initialization before the TAILQ_FOREACH(..) loop
to ensure that the value is properly initialized before it's inserted
into the TAILQ.
MFC after: 1 week
PR: 217760
Submitted by: eugen
Sponsored by: Dell EMC Isilon
Modified:
head/contrib/bsnmp/snmpd/main.c
Modified: head/contrib/bsnmp/snmpd/main.c
==============================================================================
--- head/contrib/bsnmp/snmpd/main.c Mon Mar 13 17:58:08 2017 (r315205)
+++ head/contrib/bsnmp/snmpd/main.c Mon Mar 13 18:01:01 2017 (r315206)
@@ -765,13 +765,13 @@ trans_insert_port(struct transport *t, s
{
struct tport *p;
+ port->transport = t;
TAILQ_FOREACH(p, &t->table, link) {
if (asn_compare_oid(&p->index, &port->index) > 0) {
TAILQ_INSERT_BEFORE(p, port, link);
return;
}
}
- port->transport = t;
TAILQ_INSERT_TAIL(&t->table, port, link);
}
More information about the svn-src-all
mailing list