From e0e186091a777dfe9cf3c22deeff989f459d8db9 Mon Sep 17 00:00:00 2001 From: hubertf Date: Wed, 10 Feb 1999 00:29:21 +0000 Subject: The patch below does some minor cleanup of adventure(6): an additional use of const I missed in bin/6041; avoiding a signed/unsigned warning; marking an unused parameter as such; revoking setgid privileges (including the saved gid) rather than setuid ones; includes and function prototypes in setup.c; the string passed to err() should not end with a `.'. Together with the patch sent concerning adventure's EOF handling, this synchronises adventure(6) in NetBSD with the Linux port of the NetBSD games. Via PR 6557 by Joseph Myers --- adventure/main.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'adventure/main.c') diff --git a/adventure/main.c b/adventure/main.c index e3112c12..aa21cf8c 100644 --- a/adventure/main.c +++ b/adventure/main.c @@ -1,4 +1,4 @@ -/* $NetBSD: main.c,v 1.12 1998/09/14 09:29:08 hubertf Exp $ */ +/* $NetBSD: main.c,v 1.13 1999/02/10 00:29:21 hubertf Exp $ */ /*- * Copyright (c) 1991, 1993 @@ -48,7 +48,7 @@ __COPYRIGHT("@(#) Copyright (c) 1991, 1993\n\ #if 0 static char sccsid[] = "@(#)main.c 8.1 (Berkeley) 6/2/93"; #else -__RCSID("$NetBSD: main.c,v 1.12 1998/09/14 09:29:08 hubertf Exp $"); +__RCSID("$NetBSD: main.c,v 1.13 1999/02/10 00:29:21 hubertf Exp $"); #endif #endif /* not lint */ @@ -73,9 +73,8 @@ main(argc, argv) int rval, ll; struct text *kk; - /* adventure doesn't need setuid-ness, so, just get rid of it */ - if (setuid(getuid()) < 0) - warn("setuid"); + /* revoke setgid privileges from dm */ + setregid(getgid(), getgid()); init(); /* Initialize everything */ signal(SIGINT, trapdel); -- cgit v1.2.3-56-ge451