[PATCH] Bug in NFS

Abramo Bagnara (abramo@alsa-project.org)
Fri, 13 Jul 2001 08:23:37 +0200


This is a multi-part message in MIME format.
--------------81C1B6873E2B8874ED9E1D76
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit

I have found a bug in NFSv2.

[root@igor /tmp]# mount igor:/u u
[root@igor /tmp]# cd u
[root@igor u]# umask 000
[root@igor u]# ls -l q
ls: q: File o directory inesistente
[root@igor u]# touch q
[root@igor u]# ls -l q
-rw-r--r-- 1 root root 0 lug 13 07:56 q

This seems to be caused by use of unitialized current->fs->umask via
vfs_create called by nfsd_create.

Patch for 2.4.6 follows.

-- 
Abramo Bagnara                       mailto:abramo@alsa-project.org

Opera Unica Phone: +39.546.656023 Via Emilia Interna, 140 48014 Castel Bolognese (RA) - Italy

ALSA project http://www.alsa-project.org It sounds good! --------------81C1B6873E2B8874ED9E1D76 Content-Type: text/plain; charset=us-ascii; name="nfs.diff" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="nfs.diff"

--- linux-2.4/fs/nfsd/auth.c.~1~ Mon Jul 24 08:04:10 2000 +++ linux-2.4/fs/nfsd/auth.c Fri Jul 13 08:00:10 2001 @@ -34,6 +34,7 @@ cred->cr_groups[i] = exp->ex_anon_gid; } + current->fs->umask = 0; if (cred->cr_uid != (uid_t) -1) current->fsuid = cred->cr_uid; else

--------------81C1B6873E2B8874ED9E1D76--

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/