* [PATCH] Fix reverted floppy-device permissions
@ 2010-01-11 1:02 Bryan Kadzban
2010-01-11 9:46 ` Martin Pitt
0 siblings, 1 reply; 2+ messages in thread
From: Bryan Kadzban @ 2010-01-11 1:02 UTC (permalink / raw)
To: linux-hotplug
Commit d93fb59c50a720e2a1ace2177cc824c00ff8eed6 ("rules: set mode of
floppy device nodes to 0660") changed the create_floppy_devices -M
argument to 0660, for udev-148.
Commit 78171d9549fafdfc1e29d30915104a3ad55e116b reverted this, when it
moved the call to create_floppy_devices out to another rules file, for
udev-150.
Presumably the 0660 mode was correct, and the 0640 reversion was not
intentional; this restores 0660.
Signed-off-by: Bryan Kadzban <bryan@kadzban.is-a-geek.net>
diff --git a/extras/floppy/60-floppy.rules b/extras/floppy/60-floppy.rules
index 3123bb5..53e4a9e 100644
--- a/extras/floppy/60-floppy.rules
+++ b/extras/floppy/60-floppy.rules
@@ -1,4 +1,4 @@
# do not edit this file, it will be overwritten on update
SUBSYSTEM="block", KERNEL="fd[0-9]", ACTION="add", ATTRS{cmos}="?*", ENV{CMOS_TYPE}="$attr{cmos}", \
- RUN+="create_floppy_devices -c -t $env{CMOS_TYPE} -m %M -M 0640 -G floppy $root/%k"
+ RUN+="create_floppy_devices -c -t $env{CMOS_TYPE} -m %M -M 0660 -G floppy $root/%k"
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] Fix reverted floppy-device permissions
2010-01-11 1:02 [PATCH] Fix reverted floppy-device permissions Bryan Kadzban
@ 2010-01-11 9:46 ` Martin Pitt
0 siblings, 0 replies; 2+ messages in thread
From: Martin Pitt @ 2010-01-11 9:46 UTC (permalink / raw)
To: linux-hotplug
Hey Bryan,
Bryan Kadzban [2010-01-10 17:02 -0800]:
> Commit d93fb59c50a720e2a1ace2177cc824c00ff8eed6 ("rules: set mode of
> floppy device nodes to 0660") changed the create_floppy_devices -M
> argument to 0660, for udev-148.
>
> Commit 78171d9549fafdfc1e29d30915104a3ad55e116b reverted this, when it
> moved the call to create_floppy_devices out to another rules file, for
> udev-150.
>
> Presumably the 0660 mode was correct, and the 0640 reversion was not
> intentional; this restores 0660.
Indeed, nice catch. Applied.
Thank you!
Martin
--
Martin Pitt | http://www.piware.de
Ubuntu Developer (www.ubuntu.com) | Debian Developer (www.debian.org)
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-01-11 9:46 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-01-11 1:02 [PATCH] Fix reverted floppy-device permissions Bryan Kadzban
2010-01-11 9:46 ` Martin Pitt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).