Bug 34205 - wrong (unowned) folder permissions for `extfs.d/rpm run $NAME.rpm UPGRADE`
Summary: wrong (unowned) folder permissions for `extfs.d/rpm run $NAME.rpm UPGRADE`
Status: CLOSED WONTFIX
Alias: None
Product: Sisyphus
Classification: Development
Component: mc (show other bugs)
Version: unstable
Hardware: all Linux
: P3 minor
Assignee: Sergey Y. Afonin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-11-16 08:37 MSK by ildar
Modified: 2018-08-31 14:36 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 ildar 2017-11-16 08:37:36 MSK
после установки:
$ ls -ld /usr/share/fonts/ttf/eosrei-emojione
drwx------ 2 root root 4096 ноя 16 11:25 /usr/share/fonts/ttf/eosrei-emojione
Comment 1 viy 2017-11-16 16:32:32 MSK
не подземный ли стук часом?
у меня не воспроизвелось:

$ sudo rpm -Uvh /var/ftp/pub/Linux/ALT/Sisyphus/files/noarch/RPMS/fonts-ttf-eosrei-emojione-1.0-alt1_5.noarch.rpm 
[sudo] password for igor:
Подготовка...      ##################################################
fonts-ttf-eosrei-emojione    ##################################################
Running /usr/lib/rpm/posttrans-filetriggers
igor@ohmu ~ $ ls -ld /usr/share/fonts/ttf/eosrei-emojione
drwxr-xr-x 2 root root 4096 ноя 16 15:30 /usr/share/fonts/ttf/eosrei-emojione
Comment 2 ildar 2017-11-17 10:56:48 MSK
действительно интересно то, что проблемы нет, если делать `rpm -Uvh`, но проявляется, если запустить в p8:
> # /usr/lib/mc/extfs.d/rpm run /tmp/fonts-ttf-eosrei-emojione-1.0-alt1_5.noarch.rpm UPGRADE

Мне удалось повторить это в hasher/p8, но не в hasher/Sisyphus!
Comment 3 viy 2017-11-17 11:12:21 MSK
да. непонятно, кому баг. mc в p8?
Comment 4 ildar 2017-11-17 12:23:52 MSK
перевешал.
Бага микроскопическая, исправять необязательно.
Comment 5 Sergey Y. Afonin 2017-12-11 11:41:02 MSK
Действительно какой-то непонятный подземный стук:

+ case "$1" in
+ echo 'Upgrading "fonts-ttf-eosrei-emojione-1.0-alt1_5.noarch.rpm"'
Upgrading "fonts-ttf-eosrei-emojione-1.0-alt1_5.noarch.rpm"
+ rpm -Uvh fonts-ttf-eosrei-emojione-1.0-alt1_5.noarch.rpm
Preparing...                 ##########################################################################
fonts-ttf-eosrei-emojione    ##########################################################################
Running /usr/lib/rpm/posttrans-filetriggers
+ exit 0

То есть, extfs.d/rpm run, в итоге, запускает всё тот же "rpm -Uvh". Я пока не понимаю, почему при запуске из-под mc получаются такие права. Единственное предположение - существование какого-то дефолта, отличного от обычного, в этот момент и bug 34316. По идее, это должно бы было во всех подобных случаях вылезать.
Comment 6 Sergey Y. Afonin 2017-12-12 09:46:47 MSK
(In reply to comment #5)

После исправления bug 34316 права назначились правильно. Вопрос теперь, откуда берётся непонятное умолчаение, и почему оно отличается от того, что присутствует при запуске rpm напрямую.
Comment 7 Sergey Y. Afonin 2017-12-12 10:18:28 MSK
В extfs.d/rpm нашёлся вызов umask 077. Вопрос только теперь, хорошо это, или плохо. И зачем было сделано. Если его убрать, получается 

# ls -ld /usr/share/fonts/ttf/eosrei-emojione
drwxr-xr-x 2 root root 4096 Dec 12 11:15 /usr/share/fonts/ttf/eosrei-emojione

для пакета с неисправленным bug 34316.
Comment 8 Sergey Y. Afonin 2018-04-14 10:54:00 MSK
Что-то совсем древнее...

commit 549fecfff7ab37acc2156e9bc58e427ce96660b1
Author: Norbert Warmuth <nwarmuth@privat.circular.de>
Date:   Wed Dec 16 06:16:13 1998 +0000

    Wed Dec 16 06:47:47 1998  Norbert Warmuth  <nwarmuth@privat.circular.de>

    * screen.c (Xtry_to_select): Don't select a similar file when
    "name" can't be found in the panel (when deleting files the
    selection jumpped from the deleted "file" to a directory which
    started with the same character as "file", annoying).
    Strip known vfs suffixes from "name" before trying to select
    (I think Timur made this suggestion a few months ago). Know the
    vfs is mature enough to do this.

    * vfs/vfs.c (vfs_strip_suffix_from_filename): New function which strips
    known vfs suffixes from a filename and returns a malloced string
    which has to be freed. Possible improvement: strip vfs suffix from
    last path component.

    * vfs/extfs/*: added "umask 077" to every script.
Comment 9 Sergey Y. Afonin 2018-08-31 14:36:16 MSK
(In reply to comment #7)

> В extfs.d/rpm нашёлся вызов umask 077. Вопрос только теперь, хорошо это, или
> плохо. И зачем было сделано.

Если у кого-то будут идеи, почему это надо убрать, пишите и переоткрывайте. Пока оставлю, как есть, так как проблема проявилась из-за установки из rpm неупакованного в пакет безхозного каталога, что, в общем-то, само по себе ошибка. Если кто-то придумает, зачем вызов umask 077 мог быть добавлен, тоже напишите на всякий случай.