On Jul 21, 2011, at 1:40 AM, NeilBrown wrote:
>
> Subject: [PATCH] Statd should always 'chdir' to its state directory.
>
> As statd can be started by 'mount' which can sometimes be run by a
> normal user, the current-working-directory could be anything. In
> partcular it could be in a mounted filesystem. As 'statd' continues
> running as a daemon it could keep prevent that filesystem from being
> unmounted.
>
> statd does currently 'chdir' to the state directory, but only if the
> state directory is not owned by root. This is wrong - it should check
> for root after the chdir, not before.
>
> So swap the two if statements around.
Seems reasonable.
>
> Signed-off-by: NeilBrown <neilb@xxxxxxx>
>
> --
> looks like this regression slipped in around January 2010 with the big libnsm clean up.
> -NB
>
>
> diff --git a/support/nsm/file.c b/support/nsm/file.c
> index 98b47bf..a12c753 100644
> --- a/support/nsm/file.c
> +++ b/support/nsm/file.c
> @@ -395,18 +395,18 @@ nsm_drop_privileges(const int pidfd)
> return false;
> }
>
> - if (st.st_uid == 0) {
> - xlog_warn("Running as root. "
> - "chown %s to choose different user", nsm_base_dirname);
> - return true;
> - }
> -
> if (chdir(nsm_base_dirname) == -1) {
> xlog(L_ERROR, "Failed to change working directory to %s: %m",
> nsm_base_dirname);
> return false;
> }
>
> + if (st.st_uid == 0) {
> + xlog_warn("Running as root. "
> + "chown %s to choose different user", nsm_base_dirname);
> + return true;
> + }
> +
> /*
> * If the pidfile happens to reside on NFS, dropping privileges
> * will probably cause us to lose access, even though we are
--
Chuck Lever
chuck[dot]lever[at]oracle[dot]com
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
