[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Ignore EPERM when modifying kern.geom.debugflags
From: |
Vladimir 'φ-coder/phcoder' Serbinenko |
Subject: |
Re: [PATCH] Ignore EPERM when modifying kern.geom.debugflags |
Date: |
Sat, 18 Jan 2014 16:20:33 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Icedove/24.2.0 |
go ahead
On 17.01.2014 03:55, Colin Watson wrote:
> Many tests fail when run as a non-root user on FreeBSD. The failures
> all amount to an inability to open files using grub_util_fd_open,
> because we cannot set the kern.geom.debugflags sysctl. This sysctl is
> indeed important to allow us to do such things as installing GRUB to the
> MBR, but if we need to do that and can't then we will get an error
> later. Enforcing it here is unnecessary and prevents otherwise
> perfectly reasonable operations.
> ---
> grub-core/osdep/freebsd/hostdisk.c | 12 ++++++++++--
> 1 file changed, 10 insertions(+), 2 deletions(-)
>
> diff --git a/grub-core/osdep/freebsd/hostdisk.c
> b/grub-core/osdep/freebsd/hostdisk.c
> index eb202dc..6145d07 100644
> --- a/grub-core/osdep/freebsd/hostdisk.c
> +++ b/grub-core/osdep/freebsd/hostdisk.c
> @@ -102,8 +102,16 @@ grub_util_fd_open (const char *os_dev, int flags)
> if (! (sysctl_oldflags & 0x10)
> && sysctlbyname ("kern.geom.debugflags", NULL , 0, &sysctl_flags,
> sysctl_size))
> {
> - grub_error (GRUB_ERR_BAD_DEVICE, "cannot set flags of sysctl
> kern.geom.debugflags");
> - return GRUB_UTIL_FD_INVALID;
> + if (errno == EPERM)
> + /* Running as an unprivileged user; don't worry about restoring
> + flags, although if we try to write to anything interesting such
> + as the MBR then we may fail later. */
> + sysctl_oldflags = 0x10;
> + else
> + {
> + grub_error (GRUB_ERR_BAD_DEVICE, "cannot set flags of sysctl
> kern.geom.debugflags");
> + return GRUB_UTIL_FD_INVALID;
> + }
> }
>
> ret = open (os_dev, flags, S_IROTH | S_IRGRP | S_IRUSR | S_IWUSR);
>
signature.asc
Description: OpenPGP digital signature