diff options
| author | Henning Brauer <henning@cvs.openbsd.org> | 2006-05-23 12:04:29 +0000 |
|---|---|---|
| committer | Henning Brauer <henning@cvs.openbsd.org> | 2006-05-23 12:04:29 +0000 |
| commit | bd0a71a1145af0072b689da3178830cdf67539d7 (patch) | |
| tree | 2b1ef3bed9b65ac143d4cd3a3ff8a1f7896c3750 /sbin | |
| parent | 78a58d3ad01ff821539fa7358bc2d9fc51bf130c (diff) | |
member interfaces of groups might have no IPs and ifa_lookup retun NULL,
account for that. fixes PR5130, ok dhartmei
Diffstat (limited to 'sbin')
| -rw-r--r-- | sbin/pfctl/pfctl_parser.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/sbin/pfctl/pfctl_parser.c b/sbin/pfctl/pfctl_parser.c index a00e74b3f9d..5bf93e67c2c 100644 --- a/sbin/pfctl/pfctl_parser.c +++ b/sbin/pfctl/pfctl_parser.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pfctl_parser.c,v 1.223 2006/03/21 03:31:09 dhartmei Exp $ */ +/* $OpenBSD: pfctl_parser.c,v 1.224 2006/05/23 12:04:28 henning Exp $ */ /* * Copyright (c) 2001 Daniel Hartmeier @@ -1225,7 +1225,8 @@ ifa_grouplookup(const char *ifa_name, int flags) for (ifg = ifgr.ifgr_groups; ifg && len >= sizeof(struct ifg_req); ifg++) { len -= sizeof(struct ifg_req); - n = ifa_lookup(ifg->ifgrq_member, flags); + if ((n = ifa_lookup(ifg->ifgrq_member, flags)) == NULL) + continue; if (h == NULL) h = n; else { |