From: deraadt <deraadt@openbsd.org>
Date: Sat, 10 Oct 2015 19:19:46 +0000 (+0000)
Subject: since kdump may getprotobynumber() late, do not drop "rpath".  We could
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=a5a6da98fe4efde2ecb6f297619bb90a084afa5c;p=openbsd

since kdump may getprotobynumber() late, do not drop "rpath".  We could
potentially modify pledge() to permit /etc/protocols (/etc/rpc?
/etc/services? etc) without requiring a rpath attribute.. but where would
we draw the line for what /etc files libc functions need?  At present, we
draw that line closer to the minimum.
issue found by theo@math.ethz.ch
---

diff --git a/usr.bin/kdump/kdump.c b/usr.bin/kdump/kdump.c
index a9257581bca..5fa69cdaaef 100644
--- a/usr.bin/kdump/kdump.c
+++ b/usr.bin/kdump/kdump.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: kdump.c,v 1.113 2015/10/09 01:37:08 deraadt Exp $	*/
+/*	$OpenBSD: kdump.c,v 1.114 2015/10/10 19:19:46 deraadt Exp $	*/
 
 /*-
  * Copyright (c) 1988, 1993
@@ -250,9 +250,6 @@ main(int argc, char *argv[])
 	if (!freopen(tracefile, "r", stdin))
 		err(1, "%s", tracefile);
 
-	if (pledge("stdio getpw", NULL) == -1)
-		err(1, "pledge");
-
 	if (fread_tail(&ktr_header, sizeof(struct ktr_header), 1) == 0 ||
 	    ktr_header.ktr_type != htobe32(KTR_START))
 		errx(1, "%s: not a dump", tracefile);