|
 |
011fe81 |
From 97fca4331e317b0491912c3cf3d02a9fac96c419 Mon Sep 17 00:00:00 2001
|
|
|
78a3d7d |
From: Colin Watson <cjwatson@ubuntu.com>
|
|
|
78a3d7d |
Date: Fri, 17 Jan 2014 02:28:46 +0000
|
|
|
4dcaf21 |
Subject: [PATCH 035/143] Ignore EPERM when modifying kern.geom.debugflags
|
|
|
78a3d7d |
|
|
|
78a3d7d |
Many tests fail when run as a non-root user on FreeBSD. The failures
|
|
|
78a3d7d |
all amount to an inability to open files using grub_util_fd_open,
|
|
|
78a3d7d |
because we cannot set the kern.geom.debugflags sysctl. This sysctl is
|
|
|
78a3d7d |
indeed important to allow us to do such things as installing GRUB to the
|
|
|
78a3d7d |
MBR, but if we need to do that and can't then we will get an error
|
|
|
78a3d7d |
later. Enforcing it here is unnecessary and prevents otherwise
|
|
|
78a3d7d |
perfectly reasonable operations.
|
|
|
78a3d7d |
---
|
|
|
78a3d7d |
ChangeLog | 7 +++++++
|
|
|
78a3d7d |
grub-core/osdep/freebsd/hostdisk.c | 12 ++++++++++--
|
|
|
78a3d7d |
2 files changed, 17 insertions(+), 2 deletions(-)
|
|
|
78a3d7d |
|
|
|
78a3d7d |
diff --git a/ChangeLog b/ChangeLog
|
|
|
011fe81 |
index 4688ff4..10abfe2 100644
|
|
|
78a3d7d |
--- a/ChangeLog
|
|
|
78a3d7d |
+++ b/ChangeLog
|
|
|
78a3d7d |
@@ -1,3 +1,10 @@
|
|
|
78a3d7d |
+2014-01-19 Colin Watson <cjwatson@ubuntu.com>
|
|
|
78a3d7d |
+
|
|
|
78a3d7d |
+ * grub-core/osdep/freebsd/hostdisk.c (grub_util_fd_open): Ignore
|
|
|
78a3d7d |
+ EPERM when modifying kern.geom.debugflags. It is only a problem for
|
|
|
78a3d7d |
+ such things as installing GRUB to the MBR, in which case there'll be
|
|
|
78a3d7d |
+ an error later anyway, not for opening files during tests.
|
|
|
78a3d7d |
+
|
|
|
78a3d7d |
2014-01-18 Andrey Borzenkov <arvidjaar@gmail.com>
|
|
|
78a3d7d |
|
|
|
78a3d7d |
* grub-core/Makefile.am: Build grub_emu_init.[ch] from MODULE_FILES
|
|
|
78a3d7d |
diff --git a/grub-core/osdep/freebsd/hostdisk.c b/grub-core/osdep/freebsd/hostdisk.c
|
|
|
78a3d7d |
index eb202dc..6145d07 100644
|
|
|
78a3d7d |
--- a/grub-core/osdep/freebsd/hostdisk.c
|
|
|
78a3d7d |
+++ b/grub-core/osdep/freebsd/hostdisk.c
|
|
|
78a3d7d |
@@ -102,8 +102,16 @@ grub_util_fd_open (const char *os_dev, int flags)
|
|
|
78a3d7d |
if (! (sysctl_oldflags & 0x10)
|
|
|
78a3d7d |
&& sysctlbyname ("kern.geom.debugflags", NULL , 0, &sysctl_flags, sysctl_size))
|
|
|
78a3d7d |
{
|
|
|
78a3d7d |
- grub_error (GRUB_ERR_BAD_DEVICE, "cannot set flags of sysctl kern.geom.debugflags");
|
|
|
78a3d7d |
- return GRUB_UTIL_FD_INVALID;
|
|
|
78a3d7d |
+ if (errno == EPERM)
|
|
|
78a3d7d |
+ /* Running as an unprivileged user; don't worry about restoring
|
|
|
78a3d7d |
+ flags, although if we try to write to anything interesting such
|
|
|
78a3d7d |
+ as the MBR then we may fail later. */
|
|
|
78a3d7d |
+ sysctl_oldflags = 0x10;
|
|
|
78a3d7d |
+ else
|
|
|
78a3d7d |
+ {
|
|
|
78a3d7d |
+ grub_error (GRUB_ERR_BAD_DEVICE, "cannot set flags of sysctl kern.geom.debugflags");
|
|
|
78a3d7d |
+ return GRUB_UTIL_FD_INVALID;
|
|
|
78a3d7d |
+ }
|
|
|
78a3d7d |
}
|
|
|
78a3d7d |
|
|
|
78a3d7d |
ret = open (os_dev, flags, S_IROTH | S_IRGRP | S_IRUSR | S_IWUSR);
|
|
|
78a3d7d |
--
|
|
|
37b39b7 |
1.9.3
|
|
|
78a3d7d |
|