Opened 17 years ago

Closed 16 years ago

#2076 closed task (fixed)

Inconsistent permissions for floppy devices

Reported by: alexander@… Owned by: DJ Lucas
Priority: normal Milestone: 6.4
Component: Book Version: 6.3
Severity: normal Keywords:
Cc:

Description

/dev/fd0 has mode 0660, while /dev/fd0u1440 has mode 0640. Please change the create_floppy_devices udev rule by changing 0640 to 0660.

Change History (7)

comment:1 by alexander@…, 17 years ago

Or better, remove "-M 0640".

comment:2 by bryan@linuxfromscratch.org, 16 years ago

Looking through more old tickets; I forgot this one existed. Upstream's rules still use "-M 0640" on the script, and don't specify a mode for fd[0-9]*, so there's still a mismatch. (Even though we've removed this stuff from the LFS custom rules files, and are using all upstream rules for floppy devices.)

I think the best fix is to get upstream to either add MODE="0640" to the fd[0-9]* rule, or remove "-M 0640" from the create_floppy_devices script call, as you said. (It depends on whether you want the floppy group to have raw write access to the device. I believe there have been cases where upstream didn't want e.g. the disk group to have write access to hard-drive and partition devices.) Do you want to bug them about it, or should I? (There are days where I think some of the maintainers have kill-filed me, so if I do it, nothing may happen. :-) But whatever; I can try.)

comment:3 by alexander@…, 16 years ago

The discrepancy between "disk" and "floppy" groups are because of their different purposes: "disk" is almost strictly for backup software, "floppy" is for people. Compare with CD-ROMs. So I insist in 0660 permissions for floppies.

comment:4 by bdubbs@…, 16 years ago

Milestone: 7.06.4

comment:6 by bdubbs@…, 16 years ago

Looking at udev-130, the rules are:

KERNEL=="fd[0-9]",    GROUP="floppy"
KERNEL=="fd[0-9]", ACTION=="add", ATTRS{cmos}=="?*", RUN+="create_floppy_devices -c -t $attr{cmos} -m %M -M 0640 -G floppy $root/%k"

I am reluctant to change the standard rules because they could be overwritten with a later update. It looks like we will need to add

KERNEL=="fd[0-9]", ACTION=="add", ATTRS{cmos}=="?*", RUN+="create_floppy_devices -c -t $attr{cmos} -m %M -M 0660 -G floppy $root/%k"

This should work because create_floppy_devices does an unlink of the node before recreating the new one.

comment:7 by DJ Lucas, 16 years ago

Owner: changed from lfs-book@… to DJ Lucas
Status: newassigned

comment:8 by DJ Lucas, 16 years ago

Resolution: fixed
Status: assignedclosed

Fixed in r8663.

Note: See TracTickets for help on using tickets.