diff options
author | Luiz Otavio O Souza <luiz@netgate.com> | 2015-09-15 15:12:52 -0500 |
---|---|---|
committer | Luiz Otavio O Souza <luiz@netgate.com> | 2015-10-20 12:00:04 -0500 |
commit | 850e148cc4be95b3f1ab4e6f978b4281357ead7d (patch) | |
tree | 4ba10d3a11c4693a1e611ac912e52829f0095c68 /sys/netipsec/key.c | |
parent | d86ce4afc343578198dbe5806590d5b4b009691e (diff) | |
download | FreeBSD-src-850e148cc4be95b3f1ab4e6f978b4281357ead7d.zip FreeBSD-src-850e148cc4be95b3f1ab4e6f978b4281357ead7d.tar.gz |
MFC r277797:
key_spdget uses key_setdumpsp() without SPTREE_RLOCK held (it uses
referenced pointer to sp). Remove SPTREE_RLOCK_ASSERT from
key_setdumpsp() to fix wrong assertion.
Reported by: Emeric POUPON
Obtained from: Yandex LLC
Sponsored by: Yandex LLC
TAG: IPSEC-HEAD
Issue: #4841
Diffstat (limited to 'sys/netipsec/key.c')
-rw-r--r-- | sys/netipsec/key.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/sys/netipsec/key.c b/sys/netipsec/key.c index 010c539..a3df5b5 100644 --- a/sys/netipsec/key.c +++ b/sys/netipsec/key.c @@ -2418,8 +2418,6 @@ key_setdumpsp(struct secpolicy *sp, u_int8_t type, u_int32_t seq, struct mbuf *result = NULL, *m; struct seclifetime lt; - SPTREE_RLOCK_ASSERT(); - m = key_setsadbmsg(type, 0, SADB_SATYPE_UNSPEC, seq, pid, sp->refcnt); if (!m) goto fail; |