diff options
author | Henning Brauer <henning@cvs.openbsd.org> | 2010-09-21 10:37:34 +0000 |
---|---|---|
committer | Henning Brauer <henning@cvs.openbsd.org> | 2010-09-21 10:37:34 +0000 |
commit | 70e8cfe7a19f4d46a9b0c9695144814b13f18c7e (patch) | |
tree | 03e370a3963b1abc2386a18a1ce377f671384740 | |
parent | 646fa635c1d484253f62f484c34a91e5fe693d26 (diff) |
do not increment states_tot on match rules. it is purely cosmetic, this
value is used in verbose "show rules" output as "states creations", but
match rules never create state. states_cur aka "States" is needed and
correct, match rules are being associated with states. ok ryan
-rw-r--r-- | sys/net/pf.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/sys/net/pf.c b/sys/net/pf.c index a1bc79a8d7f..e5607fb4c3c 100644 --- a/sys/net/pf.c +++ b/sys/net/pf.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pf.c,v 1.700 2010/09/21 07:04:24 henning Exp $ */ +/* $OpenBSD: pf.c,v 1.701 2010/09/21 10:37:33 henning Exp $ */ /* * Copyright (c) 2001 Daniel Hartmeier @@ -281,10 +281,8 @@ enum { PF_ICMP_MULTI_NONE, PF_ICMP_MULTI_SOLICITED, PF_ICMP_MULTI_LINK }; s->anchor.ptr->states_cur++; \ s->anchor.ptr->states_tot++; \ } \ - SLIST_FOREACH(mrm, &s->match_rules, entry) { \ + SLIST_FOREACH(mrm, &s->match_rules, entry) \ mrm->r->states_cur++; \ - mrm->r->states_tot++; \ - } \ } while (0) #define STATE_DEC_COUNTERS(s) \ |