summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorVegard Nossum <vegard.nossum@oracle.com>2016-07-27 08:44:15 +0200
committerSteffen Klassert <steffen.klassert@secunet.com>2016-07-27 13:09:00 +0200
commit7677c7560c3e80fde08a7e710d378dedabf950c3 (patch)
treefa52db2f6e1f47cbaab2dfa7569e33fa49eda652 /net
parent73efc3245fd3edb3632d82a3a9c5d5d975a02efc (diff)
downloadop-kernel-dev-7677c7560c3e80fde08a7e710d378dedabf950c3.zip
op-kernel-dev-7677c7560c3e80fde08a7e710d378dedabf950c3.tar.gz
xfrm: get rid of another incorrect WARN
During fuzzing I regularly run into this WARN(). According to Herbert Xu, this "certainly shouldn't be a WARN, it probably shouldn't print anything either". Cc: Stephen Hemminger <stephen@networkplumber.org> Cc: Steffen Klassert <steffen.klassert@secunet.com> Cc: Herbert Xu <herbert@gondor.apana.org.au> Signed-off-by: Vegard Nossum <vegard.nossum@oracle.com> Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
Diffstat (limited to 'net')
-rw-r--r--net/xfrm/xfrm_user.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/net/xfrm/xfrm_user.c b/net/xfrm/xfrm_user.c
index 1a4f142..cb65d91 100644
--- a/net/xfrm/xfrm_user.c
+++ b/net/xfrm/xfrm_user.c
@@ -2051,9 +2051,6 @@ static int xfrm_add_pol_expire(struct sk_buff *skb, struct nlmsghdr *nlh,
if (up->hard) {
xfrm_policy_delete(xp, p->dir);
xfrm_audit_policy_delete(xp, 1, true);
- } else {
- // reset the timers here?
- WARN(1, "Don't know what to do with soft policy expire\n");
}
km_policy_expired(xp, p->dir, up->hard, nlh->nlmsg_pid);
OpenPOWER on IntegriCloud