Re: [PATCH] Dont force umask to 022 since this breaks the -o umask option.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tuesday 24 April 2007, Karel Zak wrote:
>  I don't understand what do you want to fix by this patch. Please,
>  try to describe the problem.
>
>  We have to overwrite the umask, because default for mount(2) is the
>  umask of the current process. The mount & umount are suid and I don't
>  think we want to allow non-root users to control umask for mounted
>  filesystems (when umask= is not defined in fstab together with
>  user/users option).

the umask() force there breaks the umask= option in /etc/fstab ... for full 
info/test case/etc... see this bug report:
http://bugs.gentoo.org/93671
perhaps we took the wrong approach in Gentoo, but there is a bug here ;)

maybe need to load the umask option from user/fstab and then clear explicit 
bits based on the user doing the mount ...
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.


[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux