]> git.decadent.org.uk Git - nfs-utils.git/commit
Ensure statd gets started if required when non-root
authorNeil Brown <neilb@suse.de>
Wed, 26 Nov 2008 17:01:06 +0000 (12:01 -0500)
committerSteve Dickson <steved@redhat.com>
Wed, 26 Nov 2008 17:01:06 +0000 (12:01 -0500)
commit33bbeabb40d11a59266e0702adaa6a2e0acb6382
tree41b48011ee5ef732d36cd080e7e45acec4e1372d
parent068ea89e7d335d381276a2fff73d5abbb2b0a04d
Ensure statd gets started if required when non-root
user mounts an NFS filesystem.

The first time an NFS filesystem is mounted, we start statd from
/sbin/mount.nfs.  If this first time is a non-root user doing the
 mount, (thanks to e.g.  the 'users' option in /etc/fstab)
then we need to be sure that the 'setuid' status from mount.nfs
is inherited through to rpc.statd so that it runs as root.

There are two places where we loose our setuid status due to the shell
(/bin/sh) discarding.

1/ mount.nfs uses "system" to run /usr/sbin/start-statd.  This runs a
   shell which is likely to drop privileges.  So change that code to use
  'fork' and 'execl' explicitly.
2/ start-statd is a shell script.  To convince the shell to allow the
  program to run in privileged mode, we need to add a "-p" flag.

We could just call setuid(getuid()) at some appropriate time, and it
might be worth doing that as well, however I think that getting
rid of 'system()' is a good idea and once that is done, the
adding of '-p' is trivial and sufficient.

Signed-off-by: Neil Brown <neilb@suse.de>
Signed-off-by: Steve Dickson <steved@redhat.com>
utils/mount/network.c
utils/statd/start-statd