svn commit: r241846 - in head/games: bcd caesar

Eitan Adler eadler at FreeBSD.org
Mon Oct 22 03:06:54 UTC 2012


Author: eadler
Date: Mon Oct 22 03:06:53 2012
New Revision: 241846
URL: http://svn.freebsd.org/changeset/base/241846

Log:
  bcd isn't installed setuid.
  caesar isn't installed setuid.
  
  For the archaeologists:
  it seems this call originally came from NetBSD PR 5970 and 5945
  
  PR:		172566
  Submitted by:	Erik Cederstrand <erik at cederstrand.dk>
  Approved by:	cperciva
  MFC after:	3 days

Modified:
  head/games/bcd/bcd.c
  head/games/caesar/caesar.c

Modified: head/games/bcd/bcd.c
==============================================================================
--- head/games/bcd/bcd.c	Mon Oct 22 03:00:43 2012	(r241845)
+++ head/games/bcd/bcd.c	Mon Oct 22 03:06:53 2012	(r241846)
@@ -129,9 +129,6 @@ main(int argc, char **argv)
 {
 	char cardline[80];
 
-	/* revoke setgid privileges */
-	setgid(getgid());
-
 	/*
 	 * The original bcd prompts with a "%" when reading from stdin,
 	 * but this seems kind of silly.  So this one doesn't.

Modified: head/games/caesar/caesar.c
==============================================================================
--- head/games/caesar/caesar.c	Mon Oct 22 03:00:43 2012	(r241845)
+++ head/games/caesar/caesar.c	Mon Oct 22 03:06:53 2012	(r241846)
@@ -82,9 +82,6 @@ main(int argc, char **argv)
 	char *inbuf;
 	int obs[26], try, winner;
 
-	/* revoke setgid privileges */
-	setgid(getgid());
-
 	if (argc > 1)
 		printit(argv[1]);
 


More information about the svn-src-head mailing list