Решено: setuid-бит не работает со скриптами (скрипт для монтирования образов создан)

[watashiwa_daredeska]

1. Нужно указывать полный путь до su.
2. Зачем su, если Вы уже пользуетесь sudo? sudo от рута не спрашивает пароль и не требует экранирования, которое делается несколько сложнее, чем Вы написали (это тоже security flaw, кстати).
3. Любой пользователь может отмонтировать то, что примонтировал другой пользователь. Это так задумано?


P.S. Это уже не к security, а к стилю: скрипты админа принято складывать в /usr/local/bin. /usr/bin — для руления менеджером пакетов.

[/dev/random]

3. Любой пользователь может отмонтировать то, что примонтировал другой пользователь. Это так задумано?

Более того, если кто-то что-то попытается примонтировать, то, что примонтировал другой пользователь, отмонтируется. Одновременно работающие пользователи будут мешать друг другу. Это тоже задумано?

[watashiwa_daredeska]

if [ "$2" = rw ]; then

Лучше писать if [ rw = "$2" ]; then, ибо "$2" может и с "-" начинаться.

[drBatty]

Лучше писать if [ rw = "$2" ]; then, ибо "$2" может и с "-" начинаться.

почему?

Код: Выделить всё

$ [ "-w" = rw ]; echo $?
1

[watashiwa_daredeska]

почему?

потому что не все реализации такие заботливые, как GNUтая. :)

[math]

3. Любой пользователь может отмонтировать то, что примонтировал другой пользователь. Это так задумано?

Более того, если кто-то что-то попытается примонтировать, то, что примонтировал другой пользователь, отмонтируется. Одновременно работающие пользователи будут мешать друг другу. Это тоже задумано?

Да. Точка монтирования-то одна. Если с этим возникнут проблемы, тогда буду решать. Но это вряд-ли - на компе 2 юзера - я и testuser и когда они работают одновременно они друг о друге знают.

По поводу /usr/local/bin. Мне кажется для слаки правильнее оставить /usr/local для тестовых установок из исходников, перед созданием пакета. Аргумент, что /usr/bin - только для менеджера пакетов в этом случае не проходит, так как админа вполне можно считать за менеджера пакетов, а все "левые" файлы - входящими в один большой условный пакет "local_admin.txz"

По поводу "rw" = "$2". Тогда уж лучше ".$2" = ".rw". Пожалуй сделаю так для универсальности.