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

Bug#770033: partman-efi: Should force umask in mount options



Control: tag -1 +pending

Thanks for the patch - applied in git now.

On Tue, Nov 18, 2014 at 08:53:34AM -0500, Marc Deslauriers wrote:
>Package: partman-efi
>Version: 25
>Severity: normal
>Tags: patch
>User: ubuntu-devel@lists.ubuntu.com
>Usertags: origin-ubuntu vivid ubuntu-patch
>
>
>
>*** /tmp/tmpryAK4b/bug_body
>
>In Ubuntu, the attached patch was applied to achieve the following:
>
>
>  * fstab.d/efi: force umask in mount options to ensure directory never
>    ends up with incorrect permissions. (LP: #1390183)
>
>
>Thanks for considering the patch.
>
>
>-- System Information:
>Debian Release: jessie/sid
>  APT prefers utopic-updates
>  APT policy: (500, 'utopic-updates'), (500, 'utopic-security'), (500, 'utopic-proposed'), (500, 'utopic'), (100, 'utopic-backports')
>Architecture: amd64 (x86_64)
>Foreign Architectures: i386
>
>Kernel: Linux 3.16.0-25-generic (SMP w/4 CPU cores)
>Locale: LANG=en_CA.UTF-8, LC_CTYPE=en_CA.UTF-8 (charmap=UTF-8)
>Shell: /bin/sh linked to /bin/dash

>diff -Nru partman-efi-25ubuntu6/debian/changelog partman-efi-25ubuntu7/debian/changelog
>diff -Nru partman-efi-25ubuntu6/fstab.d/efi partman-efi-25ubuntu7/fstab.d/efi
>--- partman-efi-25ubuntu6/fstab.d/efi	2010-09-03 08:40:28.000000000 -0400
>+++ partman-efi-25ubuntu7/fstab.d/efi	2014-11-18 08:38:31.000000000 -0500
>@@ -23,7 +23,7 @@
> 		[ -f "$id/method" ] || continue
> 		method=$(cat $id/method)
> 		[ "$method" = efi ] || continue
>-		echo "$path" /boot/efi vfat defaults 0 1
>+		echo "$path" /boot/efi vfat umask=0077 0 1
> 		seen_efi=1
> 	done
> 	close_dialog

-- 
Steve McIntyre, Cambridge, UK.                                steve@einval.com
Welcome my son, welcome to the machine.


Reply to: