diff options
| author | Ryan Thomas McBride <mcbride@cvs.openbsd.org> | 2003-01-02 11:35:00 +0000 |
|---|---|---|
| committer | Ryan Thomas McBride <mcbride@cvs.openbsd.org> | 2003-01-02 11:35:00 +0000 |
| commit | 71255e0a590e820c79449be3280ecadfbab0f623 (patch) | |
| tree | 63f07a7ecba420f11fbdec19e4a7522d596b7e58 /sys/net/pf_ioctl.c | |
| parent | f56aff888bca04b48bebd0af601992c6c7d8c20c (diff) | |
Require a direction to be specified for rules which do routing.
ok dhartmei@ henning@
Diffstat (limited to 'sys/net/pf_ioctl.c')
| -rw-r--r-- | sys/net/pf_ioctl.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/sys/net/pf_ioctl.c b/sys/net/pf_ioctl.c index 828443a7985..b520be303ec 100644 --- a/sys/net/pf_ioctl.c +++ b/sys/net/pf_ioctl.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pf_ioctl.c,v 1.39 2003/01/01 16:07:01 henning Exp $ */ +/* $OpenBSD: pf_ioctl.c,v 1.40 2003/01/02 11:34:59 mcbride Exp $ */ /* * Copyright (c) 2001 Daniel Hartmeier @@ -604,6 +604,8 @@ pfioctl(dev_t dev, u_long cmd, caddr_t addr, int flags, struct proc *p) } } + if (rule->rt && !rule->direction) + error = EINVAL; if (pf_dynaddr_setup(&rule->src.addr, rule->af)) error = EINVAL; if (pf_dynaddr_setup(&rule->dst.addr, rule->af)) @@ -823,6 +825,8 @@ pfioctl(dev_t dev, u_long cmd, caddr_t addr, int flags, struct proc *p) } else newrule->ifp = NULL; + if (newrule->rt && !newrule->direction) + error = EINVAL; if (pf_dynaddr_setup(&newrule->src.addr, newrule->af)) error = EINVAL; if (pf_dynaddr_setup(&newrule->dst.addr, newrule->af)) |