Bug 26294 - directory mode should be 755, not 555
Summary: directory mode should be 755, not 555
Status: REOPENED
Alias: None
Product: Sisyphus
Classification: Development
Component: kernel-image-std-def (show other bugs)
Version: unstable
Hardware: all Linux
: P3 enhancement
Assignee: Vitaly Chikunov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks: 26221
  Show dependency tree
 
Reported: 2011-09-14 10:59 MSK by Michael Shigorin
Modified: 2012-05-17 20:48 MSK (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Shigorin 2011-09-14 10:59:22 MSK
В пакете filesystem-2.3.8-alt1 права на каталог /sys заданы 0555,root,root;
в то время как в загруженной системе со смонтированным /sys они 0755 root:root
=> лишняя строчка в `rpm -Va`.  Предлагаю и в пакете поставить 755.
Comment 1 Dmitry V. Levin 2011-09-15 02:57:49 MSK
А почему, кстати, у смонтированных /sys и /selinux они 0755, если у аналогично смонтированного /proc они 0555?
Comment 2 Michael Shigorin 2011-09-15 14:21:22 MSK
Не знаю; несущественно:
# touch /sys/1
touch: cannot touch `/sys/1': No such file or directory
# _

Возможно, это для более древнего /proc права кривые, не отражающие фактического положения с возможностью записи...
Comment 3 Dmitry V. Levin 2011-09-15 15:00:34 MSK
(In reply to comment #2)
> Не знаю; несущественно:
> # touch /sys/1
> touch: cannot touch `/sys/1': No such file or directory
> # _
> 
> Возможно, это для более древнего /proc права кривые, не отражающие фактического
> положения с возможностью записи...

Продолжаю не видеть разницы между /proc и /sys в этом вопросе:
# touch /proc/foo /sys/foo
touch: cannot touch `/proc/foo': No such file or directory
touch: cannot touch `/sys/foo': No such file or directory
# rmdir /proc/sys /sys/fs
rmdir: failed to remove `/proc/sys': Operation not permitted
rmdir: failed to remove `/sys/fs': Operation not permitted

И там, и там нет возможности осуществлять запись, вне зависимости от прав доступа к /proc и /sys.  Так что вопрос остается в силе: почему на одном 0555 нормально, а на другом нужно 0755?
Comment 4 Michael Shigorin 2011-09-15 15:16:21 MSK
(In reply to comment #3)
> Так что вопрос остается в силе: почему на одном 0555
> нормально, а на другом нужно 0755?
В апстрим, всё в апстрим.  Бегло погрепал-посмотрел fs/{proc,sysfs} и соответствующие include/linux/*_fs.h -- мест явного прибития не заметил.

> Продолжаю не видеть разницы между /proc и /sys в этом вопросе:
Предлагаю пожать плечами и привести состояние пакета к наблюдаемой реальности.  Возможно, от раскапывания этой разницы какой-то толк и будет, но пока кажется, что максимум косметический.
Comment 5 Dmitry V. Levin 2011-12-13 19:29:35 MSK
$ git grep S_IFDIR fs/proc/root.c fs/sysfs/mount.c
fs/proc/root.c:	.mode		= S_IFDIR | S_IRUGO | S_IXUGO, 
fs/sysfs/mount.c:	.s_mode		= S_IFDIR | S_IRWXU | S_IRUGO | S_IXUGO,

Предлагаю пофиксить ядро, чтобы /sys по умолчанию монтировался с теми же правами 0555, что и /proc.
Comment 6 Michael Shigorin 2011-12-13 20:43:14 MSK
(In reply to comment #4)
> Предлагаю пожать плечами и привести состояние пакета к наблюдаемой реальности. 
> Возможно, от раскапывания этой разницы какой-то толк и будет, но пока кажется,
> что максимум косметический.
Дим, эта разница не стоит глупого отличия ядра от апстримного.  Если её фиксить перфекционистски, то в linux-kernel@, а в дистрибутиве предлагаю до тех пор подогнать filesystem.  Если ты это делать не намерен, тогда закрываем как WONTFIX и дело с концом.
Comment 7 Vitaly Kuznetsov 2011-12-14 19:42:04 MSK
Я отправил соответствующий патч в linux-kernel
Comment 8 Dmitry V. Levin 2012-05-17 19:16:51 MSK
Fixed in std-def and un-def kernels.
Comment 9 Michael Shigorin 2012-05-17 20:48:27 MSK
(In reply to comment #7)
> Я отправил соответствующий патч в linux-kernel
Спасибо!

(In reply to comment #8)
> Fixed in std-def and un-def kernels.
На 3.3.6-std-def-alt1 и 3.3.6-un-def-alt1 не подтверждаю:
наблюдаю 555 на /proc и 755 на /sys.

Впрочем, всё так же несущественно...