diff options
| author | Henning Brauer <henning@cvs.openbsd.org> | 2007-06-07 13:32:07 +0000 |
|---|---|---|
| committer | Henning Brauer <henning@cvs.openbsd.org> | 2007-06-07 13:32:07 +0000 |
| commit | 8f05df1780d1827016fd2ab44b37d63d9f78cdb0 (patch) | |
| tree | 803c89981958f7d58faf3a70aedb65a155e90733 /sys/net/pf_ioctl.c | |
| parent | 7b7661e6a4b8cef04cb05bf15dd9766eb94f1197 (diff) | |
PR 5502 From: Marc Huber <Marc.Huber@web.de>
pfioctl()'s DIOCKILLSTATES triggers panic due to wrong test
variable in for() loop.
well analyzed and fixed, excellent PR, applied verbatim, thanks!
(this was fallout from the state - state key split)
Diffstat (limited to 'sys/net/pf_ioctl.c')
| -rw-r--r-- | sys/net/pf_ioctl.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/net/pf_ioctl.c b/sys/net/pf_ioctl.c index 8535af9c172..f1340e7695b 100644 --- a/sys/net/pf_ioctl.c +++ b/sys/net/pf_ioctl.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pf_ioctl.c,v 1.179 2007/06/01 18:44:23 henning Exp $ */ +/* $OpenBSD: pf_ioctl.c,v 1.180 2007/06/07 13:32:06 henning Exp $ */ /* * Copyright (c) 2001 Daniel Hartmeier @@ -1577,7 +1577,7 @@ pfioctl(dev_t dev, u_long cmd, caddr_t addr, int flags, struct proc *p) struct pfioc_state_kill *psk = (struct pfioc_state_kill *)addr; int killed = 0; - for (s = RB_MIN(pf_state_tree_id, &tree_id); sk; + for (s = RB_MIN(pf_state_tree_id, &tree_id); s; s = nexts) { nexts = RB_NEXT(pf_state_tree_id, &tree_id, s); sk = s->state_key; |