diff options
| author | Damien Miller <djm@cvs.openbsd.org> | 2005-05-03 03:41:12 +0000 |
|---|---|---|
| committer | Damien Miller <djm@cvs.openbsd.org> | 2005-05-03 03:41:12 +0000 |
| commit | 5c95f5e2f59798905d4a5b67439d42ae084252e1 (patch) | |
| tree | 25eaa465c5ea04d1c9908ea6d801304b16d2d948 /usr.sbin/procmap | |
| parent | 9160bd40bfe1cf2a8155e00c62a9eeed136cfd91 (diff) | |
setresgid; ok deraadt@
Diffstat (limited to 'usr.sbin/procmap')
| -rw-r--r-- | usr.sbin/procmap/procmap.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/usr.sbin/procmap/procmap.c b/usr.sbin/procmap/procmap.c index 3c199936552..de59a806eb8 100644 --- a/usr.sbin/procmap/procmap.c +++ b/usr.sbin/procmap/procmap.c @@ -1,4 +1,4 @@ -/* $OpenBSD: procmap.c,v 1.18 2005/03/25 16:54:17 jaredy Exp $ */ +/* $OpenBSD: procmap.c,v 1.19 2005/05/03 03:41:11 djm Exp $ */ /* $NetBSD: pmap.c,v 1.1 2002/09/01 20:32:44 atatat Exp $ */ /* @@ -206,6 +206,7 @@ main(int argc, char *argv[]) struct kinfo_proc *kproc; /* struct proc proc; */ char *kmem, *kernel; + gid_t gid; pid = -1; verbose = debug = 0; @@ -261,10 +262,10 @@ main(int argc, char *argv[]) * Discard setgid privileges if not the running kernel so that bad * guys can't print interesting stuff from kernel memory. */ - if (kernel != NULL || kmem != NULL) { - setegid(getgid()); - setgid(getgid()); - } + gid = getgid(); + if (kernel != NULL || kmem != NULL) + if (setresgid(gid, gid, gid) == -1) + err(1, "setresgid"); argc -= optind; argv += optind; @@ -280,8 +281,9 @@ main(int argc, char *argv[]) /* start by opening libkvm */ kd = kvm_openfiles(kernel, kmem, NULL, O_RDONLY, errbuf); - setegid(getgid()); - setgid(getgid()); + if (kernel == NULL && kmem == NULL) + if (setresgid(gid, gid, gid) == -1) + err(1, "setresgid"); if (kd == NULL) errx(1, "%s", errbuf); |