diff options
| author | Henning Brauer <henning@cvs.openbsd.org> | 2002-07-30 14:53:49 +0000 |
|---|---|---|
| committer | Henning Brauer <henning@cvs.openbsd.org> | 2002-07-30 14:53:49 +0000 |
| commit | 2f361d9caad962b10eecb49c83ae29d6cfa06cc6 (patch) | |
| tree | 3443a6ffde90b5e1b9c769a9211845ad0b28f7e3 /sbin | |
| parent | 80e26f661b15194045490a0805d3e8994b3ba682 (diff) | |
grmpf.
in some cases, on non-tcp rules flags weren't resetted. cosmetical only
problem. but, well, checking for r->flags and r->flagset if we could have
assigned them zero just one round ago is just stupid, and it's not needed to
check them at all.
ok pb@, dhartmei@
Diffstat (limited to 'sbin')
| -rw-r--r-- | sbin/pfctl/parse.y | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/sbin/pfctl/parse.y b/sbin/pfctl/parse.y index 8cda4094e87..571abd9f181 100644 --- a/sbin/pfctl/parse.y +++ b/sbin/pfctl/parse.y @@ -1,4 +1,4 @@ -/* $OpenBSD: parse.y,v 1.135 2002/07/30 09:31:05 henning Exp $ */ +/* $OpenBSD: parse.y,v 1.136 2002/07/30 14:53:48 henning Exp $ */ /* * Copyright (c) 2001 Markus Friedl. All rights reserved. @@ -1882,8 +1882,7 @@ expand_rule(struct pf_rule *r, r->type = icmp_type->type; r->code = icmp_type->code; - if (r->proto && r->proto != IPPROTO_TCP && - (r->flags || r->flagset)) { + if (r->proto && r->proto != IPPROTO_TCP) { r->flags = 0; r->flagset = 0; } else { |