Author: delphij
Date: Wed Sep 23 01:07:45 2015
New Revision: 288139
URL: https://svnweb.freebsd.org/changeset/base/288139

Log:
  MFC r287590:
  
  w(1) is not setgid binary since r53279, so remove the setgid() call.
  
  Reviewed By:  wollman

Modified:
  stable/10/usr.bin/w/Makefile
  stable/10/usr.bin/w/w.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/usr.bin/w/Makefile
==============================================================================
--- stable/10/usr.bin/w/Makefile        Wed Sep 23 00:32:38 2015        
(r288138)
+++ stable/10/usr.bin/w/Makefile        Wed Sep 23 01:07:45 2015        
(r288139)
@@ -6,8 +6,6 @@ SRCS=   fmt.c pr_time.c proc_compare.c w.c
 MAN=   w.1 uptime.1
 DPADD= ${LIBKVM} ${LIBUTIL}
 LDADD= -lkvm -lutil
-#BINGRP= kmem
-#BINMODE=2555
 LINKS= ${BINDIR}/w ${BINDIR}/uptime
 
 .PATH: ${.CURDIR}/../../bin/ps

Modified: stable/10/usr.bin/w/w.c
==============================================================================
--- stable/10/usr.bin/w/w.c     Wed Sep 23 00:32:38 2015        (r288138)
+++ stable/10/usr.bin/w/w.c     Wed Sep 23 01:07:45 2015        (r288139)
@@ -132,7 +132,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;
        char *x_suffix;
        char buf[MAXHOSTNAMELEN], errbuf[_POSIX2_LINE_MAX];
@@ -152,7 +152,6 @@ main(int argc, char *argv[])
                p = "dhiflM:N:nsuw";
        }
 
-       dropgid = 0;
        memf = _PATH_DEVNULL;
        nlistf = NULL;
        while ((ch = getopt(argc, argv, p)) != -1)
@@ -169,11 +168,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;
@@ -193,13 +190,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-stable-10@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-stable-10
To unsubscribe, send any mail to "svn-src-stable-10-unsubscr...@freebsd.org"

Reply via email to