diff options
| author | Daniel Hartmeier <dhartmei@cvs.openbsd.org> | 2001-09-27 17:49:13 +0000 |
|---|---|---|
| committer | Daniel Hartmeier <dhartmei@cvs.openbsd.org> | 2001-09-27 17:49:13 +0000 |
| commit | c5c8f3e992a12807e021c397d336fe2da0fcbefe (patch) | |
| tree | 14d9c67ec17bada49eb505b2152fc040d604439f /sys/net/pf.c | |
| parent | 13527fa356aaa4a28ed573a411ec715a61c32ff0 (diff) | |
The skip steps array was one element short (since adding steps for af).
This invoked undefined behaviour under the proper circumstances.
Diffstat (limited to 'sys/net/pf.c')
| -rw-r--r-- | sys/net/pf.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/net/pf.c b/sys/net/pf.c index 4b71d49c851..e695f369413 100644 --- a/sys/net/pf.c +++ b/sys/net/pf.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pf.c,v 1.158 2001/09/27 15:15:17 dhartmei Exp $ */ +/* $OpenBSD: pf.c,v 1.159 2001/09/27 17:49:12 dhartmei Exp $ */ /* * Copyright (c) 2001 Daniel Hartmeier @@ -2100,7 +2100,7 @@ pf_calc_skip_steps(struct pf_rulequeue *rules) r = TAILQ_FIRST(rules); while (r != NULL) { a = 0; - for (i = 0; i < 6; ++i) { + for (i = 0; i < 7; ++i) { a |= 1 << i; r->skip[i] = TAILQ_NEXT(r, entries); } |