diff options
| author |   msf <msf@openbsd.org> | 2004-07-22 23:21:10 +0000 |
|---|---|---|
| committer | msf <msf@openbsd.org> | 2004-07-22 23:21:10 +0000 |
| commit | 5a527885c3edcd4245797e674b856fa68014973f (patch) | |
| tree | 034338f251fd73f6f0bdf496aa31e486d8853284 | |
| parent | use (diff) | |
| download | wireguard-openbsd-5a527885c3edcd4245797e674b856fa68014973f.tar.xz wireguard-openbsd-5a527885c3edcd4245797e674b856fa68014973f.zip | |
Add missing check for NULL in DIOCCHANGERULE. This prevents a crash in
certain rare cases.
ok mcbride@ dhartmei@
| -rw-r--r-- | sys/net/pf_ioctl.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/sys/net/pf_ioctl.c b/sys/net/pf_ioctl.c index c8a5704e68c..0278de8c19c 100644 --- a/sys/net/pf_ioctl.c +++ b/sys/net/pf_ioctl.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pf_ioctl.c,v 1.128 2004/07/05 00:15:20 henning Exp $ */ +/* $OpenBSD: pf_ioctl.c,v 1.129 2004/07/22 23:21:10 msf Exp $ */ /* * Copyright (c) 2001 Daniel Hartmeier @@ -1420,7 +1420,8 @@ pfioctl(dev_t dev, u_long cmd, caddr_t addr, int flags, struct proc *p) while ((oldrule != NULL) && (oldrule->nr != pcr->nr)) oldrule = TAILQ_NEXT(oldrule, entries); if (oldrule == NULL) { - pf_rm_rule(NULL, newrule); + if (newrule != NULL) + pf_rm_rule(NULL, newrule); error = EINVAL; splx(s); break; |