Author: delphij
Date: Wed Sep  9 05:17:04 2015
New Revision: 287590
URL: https://svnweb.freebsd.org/changeset/base/287590

Log:
  w(1) is not setgid binary since r53279, so remove the setgid() call.
  
  Reviewed By:  wollman
  MFC after:    2 weeks
  Differential Revision: https://reviews.freebsd.org/D3541

Modified:
  head/usr.bin/w/Makefile
  head/usr.bin/w/w.c

Modified: head/usr.bin/w/Makefile
==============================================================================
--- head/usr.bin/w/Makefile     Wed Sep  9 04:34:55 2015        (r287589)
+++ head/usr.bin/w/Makefile     Wed Sep  9 05:17:04 2015        (r287590)
@@ -5,8 +5,6 @@ PROG=   w
 SRCS=  fmt.c pr_time.c proc_compare.c w.c
 MAN=   w.1 uptime.1
 LIBADD=        kvm sbuf xo util
-#BINGRP= kmem
-#BINMODE=2555
 LINKS= ${BINDIR}/w ${BINDIR}/uptime
 
 .PATH: ${.CURDIR}/../../bin/ps

Modified: head/usr.bin/w/w.c
==============================================================================
--- head/usr.bin/w/w.c  Wed Sep  9 04:34:55 2015        (r287589)
+++ head/usr.bin/w/w.c  Wed Sep  9 05:17:04 2015        (r287590)
@@ -135,7 +135,7 @@ main(int argc, char *argv[])
        struct kinfo_proc *dkp;
        struct stat *stp;
        time_t touched;
-       int ch, i, nentries, nusers, wcmd, longidle, longattime, dropgid;
+       int ch, i, nentries, nusers, wcmd, longidle, longattime;
        const char *memf, *nlistf, *p, *save_p;
        char *x_suffix;
        char buf[MAXHOSTNAMELEN], errbuf[_POSIX2_LINE_MAX];
@@ -159,7 +159,6 @@ main(int argc, char *argv[])
                p = "dhiflM:N:nsuw";
        }
 
-       dropgid = 0;
        memf = _PATH_DEVNULL;
        nlistf = NULL;
        while ((ch = getopt(argc, argv, p)) != -1)
@@ -176,11 +175,9 @@ main(int argc, char *argv[])
                case 'M':
                        header = 0;
                        memf = optarg;
-                       dropgid = 1;
                        break;
                case 'N':
                        nlistf = optarg;
-                       dropgid = 1;
                        break;
                case 'n':
                        nflag = 1;
@@ -200,13 +197,6 @@ main(int argc, char *argv[])
        _res.retrans = 2;       /* resolver timeout to 2 seconds per try */
        _res.retry = 1;         /* only try once.. */
 
-       /*
-        * Discard setgid privileges if not the running kernel so that bad
-        * guys can't print interesting stuff from kernel memory.
-        */
-       if (dropgid)
-               setgid(getgid());
-
        if ((kd = kvm_openfiles(nlistf, memf, NULL, O_RDONLY, errbuf)) == NULL)
                errx(1, "%s", errbuf);
 
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to