diff options
author | mestre <mestre@openbsd.org> | 2016-03-29 14:53:27 +0000 |
---|---|---|
committer | mestre <mestre@openbsd.org> | 2016-03-29 14:53:27 +0000 |
commit | c364e0ec61546950de12bace9074d25c0382d28c (patch) | |
tree | a53070390c1b773544b9d15f319fcbf5f3f42faf /usr.sbin/authpf | |
parent | the stack checks the mac address of rxed packets, so vlan(4) doesnt have to (diff) | |
download | wireguard-openbsd-c364e0ec61546950de12bace9074d25c0382d28c.tar.xz wireguard-openbsd-c364e0ec61546950de12bace9074d25c0382d28c.zip |
- Add missing goto in order to avoid a dereference of a null object
- While here remove lint comment
OK millert@
Diffstat (limited to 'usr.sbin/authpf')
-rw-r--r-- | usr.sbin/authpf/authpf.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/usr.sbin/authpf/authpf.c b/usr.sbin/authpf/authpf.c index 44054a80b4b..7e5a1e3d6c3 100644 --- a/usr.sbin/authpf/authpf.c +++ b/usr.sbin/authpf/authpf.c @@ -1,4 +1,4 @@ -/* $OpenBSD: authpf.c,v 1.124 2015/12/08 07:11:53 mmcc Exp $ */ +/* $OpenBSD: authpf.c,v 1.125 2016/03/29 14:53:27 mestre Exp $ */ /* * Copyright (C) 1998 - 2007 Bob Beck (beck@openbsd.org). @@ -342,7 +342,6 @@ main(int argc, char *argv[]) } } - /* NOTREACHED */ dogdeath: printf("\r\n\r\nSorry, this service is currently unavailable due to "); printf("technical difficulties\r\n\r\n"); @@ -756,6 +755,7 @@ change_filter(int add, const char *luser, const char *ipsrc) if((grent = getgrgid(getgid())) == NULL) { syslog(LOG_ERR, "Group not found user %s, gid %d", luser, getgid()); + goto error; } if (luser == NULL || !luser[0] || ipsrc == NULL || !ipsrc[0]) { |