summaryrefslogtreecommitdiff
path: root/usr.sbin/authpf
diff options
context:
space:
mode:
authorBob Beck <beck@cvs.openbsd.org>2005-12-12 16:06:02 +0000
committerBob Beck <beck@cvs.openbsd.org>2005-12-12 16:06:02 +0000
commit3f311b3b863b6f8e99bbad4b2efc040d4b1e2ca0 (patch)
treef696f9a660dffc19cabd1dcba84736acb0383e67 /usr.sbin/authpf
parent2db01c69c96e70f23f573d4653f8aa4cc523b2b2 (diff)
correct err() usage and remove the do_death which is unneeded in
the child proceess, (as noticed by <evol@online.ptt.ru>)
Diffstat (limited to 'usr.sbin/authpf')
-rw-r--r--usr.sbin/authpf/authpf.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/usr.sbin/authpf/authpf.c b/usr.sbin/authpf/authpf.c
index d21abcb347d..f331dec69f3 100644
--- a/usr.sbin/authpf/authpf.c
+++ b/usr.sbin/authpf/authpf.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: authpf.c,v 1.95 2005/12/12 16:02:32 beck Exp $ */
+/* $OpenBSD: authpf.c,v 1.96 2005/12/12 16:06:01 beck Exp $ */
/*
* Copyright (C) 1998 - 2002 Bob Beck (beck@openbsd.org).
@@ -687,8 +687,7 @@ change_filter(int add, const char *luser, const char *ipsrc)
/* revoke group privs before exec */
gid = getgid();
if (setregid(gid, gid) == -1) {
- err(1, "setregid: %s", strerror(errno));
- do_death(0);
+ err(1, "setregid");
}
execvp(PATH_PFCTL, pargv);
warn("exec of %s failed", PATH_PFCTL);