diff options
author | Ed Schouten <ed@FreeBSD.org> | 2011-11-06 16:40:17 +0000 |
---|---|---|
committer | Ed Schouten <ed@FreeBSD.org> | 2011-11-06 16:40:17 +0000 |
commit | 4c46bfb6dc40ccf7d6effbef5ed28a521aea764b (patch) | |
tree | 64753ac6a2d25e5e411269d4f9ce2b18a076eb4e /usr.sbin | |
parent | acd29a915fee0ac32a191d53f975c2aef31da745 (diff) | |
download | src-4c46bfb6dc40ccf7d6effbef5ed28a521aea764b.tar.gz src-4c46bfb6dc40ccf7d6effbef5ed28a521aea764b.zip |
Eliminate global variables.
There is no reason why these three variables should be declared as
global variables, while the others aren't. They are only used inside
main().
Notes
Notes:
svn path=/head/; revision=227223
Diffstat (limited to 'usr.sbin')
-rw-r--r-- | usr.sbin/chroot/chroot.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/usr.sbin/chroot/chroot.c b/usr.sbin/chroot/chroot.c index a0122f777e66..9db0192dfe09 100644 --- a/usr.sbin/chroot/chroot.c +++ b/usr.sbin/chroot/chroot.c @@ -56,16 +56,12 @@ __FBSDID("$FreeBSD$"); static void usage(void); -char *user; /* user to switch to before running program */ -char *group; /* group to switch to ... */ -char *grouplist; /* group list to switch to ... */ - int main(int argc, char *argv[]) { struct group *gp; struct passwd *pw; - char *endp, *p; + char *endp, *p, *user, *group, *grouplist; const char *shell; gid_t gid, *gidlist; uid_t uid; @@ -74,6 +70,7 @@ main(int argc, char *argv[]) gid = 0; uid = 0; + user = group = grouplist = NULL; while ((ch = getopt(argc, argv, "G:g:u:")) != -1) { switch(ch) { case 'u': |