Bug#488865: marked as done (initramfs-tools: buggy check for read-only /boot)
Your message dated Thu, 3 Jul 2008 17:09:43 +0200
with message-id <20080703150943.GO13910@stro.at>
and subject line Re: Bug#488865: initramfs-tools: buggy check for read-only /boot
has caused the Debian Bug report #488865,
regarding initramfs-tools: buggy check for read-only /boot
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)
--
488865: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=488865
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: initramfs-tools
Version: 0.85h
Severity: normal
Tags: patch
The script /usr/sbin/update-initramfs checks if /boot is mounted read-only,
and aborts if so. Unfortunately, the test also triggers in other cases,
e.g. on option errors=continue (because this phrase contains 'ro').
A simple fix (check for 'ro' delimited by commas or the begging or the
end of options string):
--- /usr/sbin/update-initramfs.0 2008-07-01 20:31:03.000000000 +0200
+++ /usr/sbin/update-initramfs 2008-07-01 20:31:44.000000000 +0200
@@ -237,7 +237,7 @@
ro_boot_check()
{
[ -r /proc/mounts ] || return 0
- boot_opts=$(awk '/boot/{if (match($4, /ro/) && $2 == "/boot")
+ boot_opts=$(awk '/boot/{if (match($4, /(^|,)ro($|,)/) && $2 == "/boot")
print "ro"}' /proc/mounts)
if [ -n "${boot_opts}" ]; then
echo "WARNING: /boot is ro mounted."
-- System Information:
Omitted because reporting from different installation.
--- End Message ---
--- Begin Message ---
Version: 0.91e
On Tue, 01 Jul 2008, Zbyszek Szmek wrote:
> The script /usr/sbin/update-initramfs checks if /boot is mounted read-only,
> and aborts if so. Unfortunately, the test also triggers in other cases,
> e.g. on option errors=continue (because this phrase contains 'ro').
>
> A simple fix (check for 'ro' delimited by commas or the begging or the
> end of options string):
fixed in newer version, thanks for patch and report.
best regards
--
maks
--- End Message ---
Reply to: