[systemd-devel] [PATCH 02/10] rules: updates to default device permissions in 50-udev-default.rules

[Kay Sievers]

On Wed, Jul 16, 2014 at 12:09 PM, Jon Severinsson <jon at severinsson.net> wrote:
> From: Martin Pitt <martin.pitt at ubuntu.com>
>
> Taken from the previous Debian specific rules, this is the remaining difference
> over the upstream 50-udev-default.rules.
> ---
>  rules/50-udev-default.rules | 14 +++++++++++++-
>  1 file changed, 13 insertions(+), 1 deletion(-)
>
> diff --git a/rules/50-udev-default.rules b/rules/50-udev-default.rules
> index 2bf9c1e..e279285 100644
> --- a/rules/50-udev-default.rules
> +++ b/rules/50-udev-default.rules
> @@ -20,8 +20,11 @@ SUBSYSTEM=="tty", KERNEL=="ttysclp[0-9]*", GROUP="tty", MODE="0620"
>  SUBSYSTEM=="tty", KERNEL=="3270/tty[0-9]*", GROUP="tty", MODE="0620"
>  SUBSYSTEM=="vc", KERNEL=="vcs*|vcsa*", GROUP="tty"
>  KERNEL=="tty[A-Z]*[0-9]|pppox[0-9]*|ircomm[0-9]*|noz[0-9]*|rfcomm[0-9]*", GROUP="dialout"
> +KERNEL=="mISDNtimer", GROUP="dialout"
> +KERNEL=="mwave", GROUP="dialout"

This is outdated technology, which we do not support by default. It
should move into its own rules file, shipped by the package that needs
it.

>  SUBSYSTEM=="mem", KERNEL=="mem|kmem|port", GROUP="kmem", MODE="0640"
> +KERNEL=="nvram", GROUP="kmem", MODE="0640"

Which platform has that, and why is it needed? I think we just removed
that in the past.

>  SUBSYSTEM=="input", GROUP="input"
>  SUBSYSTEM=="input", KERNEL=="js[0-9]*", MODE="0664"
> @@ -54,7 +57,7 @@ SUBSYSTEM=="block", KERNEL=="sr[0-9]*", GROUP="cdrom"
>  SUBSYSTEM=="scsi_generic", SUBSYSTEMS=="scsi", ATTRS{type}=="4|5", GROUP="cdrom"
>  KERNEL=="sch[0-9]*", GROUP="cdrom"
>  KERNEL=="pktcdvd[0-9]*", GROUP="cdrom"
> -KERNEL=="pktcdvd", GROUP="cdrom"
> +KERNEL=="pktcdvd", GROUP="cdrom", MODE="0644"

This is outdated technology, which we do not support by default. It
should move into its own rules file, shipped by the package that needs
it.

>  SUBSYSTEM=="scsi_generic|scsi_tape", SUBSYSTEMS=="scsi", ATTRS{type}=="1|8", GROUP="tape"
>  SUBSYSTEM=="scsi_generic", SUBSYSTEMS=="scsi", ATTRS{type}=="0", GROUP="disk"
> @@ -71,4 +74,13 @@ KERNEL=="tun", MODE="0666", OPTIONS+="static_node=net/tun"
>
>  KERNEL=="fuse", MODE="0666", OPTIONS+="static_node=fuse"
>
> +# not the ideal group, but backwards compatibility with previous Debian versions
> +KERNEL=="lirc[0-9]*", GROUP="video"
> +
> +KERNEL=="legousbtower*", MODE="0666"
> +KERNEL=="sonypi", MODE="0666"
> +KERNEL=="mmtimer", MODE="0644"
> +KERNEL=="sgi_*", MODE="0666"
> +KERNEL=="z90crypt", MODE="0666"

We don't do that, it is just wrong.