Bug 15090 - disable shutdown from remote display
Summary: disable shutdown from remote display
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: kde-settings-kdm (show other bugs)
Version: unstable
Hardware: all Linux
: P2 minor
Assignee: Sergey V Turchin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks: 15333 14888
  Show dependency tree
 
Reported: 2008-03-26 15:41 MSK by Michael Shigorin
Modified: 2008-04-14 15:56 MSD (History)
2 users (show)

See Also:


Attachments
kdmrc package-to-altsp difference (874 bytes, patch)
2008-04-11 21:58 MSD, Michael Shigorin
no flags Details | Diff
kdmrc package-to-nicedefault difference (679 bytes, patch)
2008-04-11 21:59 MSD, Michael Shigorin
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Shigorin 2008-03-26 15:41:29 MSK
Предлагается модифицировать штатный /etc/X11/kdm/kdmrc к такому виду по части
AllowShutdown:

[X-*-Core]
AllowShutdown=None

[X-:*-Core]
AllowShutdown=Root

Вроде ничего сломать не должно (проверено на LTSP), а на терминалах не будет
смущать людей ненужная им кнопка, которая в случае последнего пользователя
выключит совсем не локальную железку :)
Comment 1 Michael Shigorin 2008-03-26 15:54:01 MSK
Ой, виноват -- конечно же,

[X-:*-Core]
AllowShutdown=All
Comment 2 Sergey V Turchin 2008-03-26 16:45:12 MSK
(In reply to comment #0)
> [X-*-Core]
> AllowShutdown=None
а может
AllowShutdown=Root
?
 
> в случае последнего пользователя
> выключит совсем не локальную железку :)
А разве позволяет?
Comment 3 Michael Shigorin 2008-03-26 16:57:33 MSK
(In reply to comment #2)
> > [X-*-Core]
> > AllowShutdown=None
> а может
> AllowShutdown=Root
> ?
Нет, тогда есть кнопочка "выключить" и куча ненужных вопросов.  Рут, на крайняк,
зайдёт юзером и скажет sudo poweroff :-)

(это мнение может измениться, когда придумаем/стырим где обёртку, чтоб
пробрасывать требование выключения на терминал -- тогда Root будет само то)

> > в случае последнего пользователя
> > выключит совсем не локальную железку :)
> А разве позволяет?
Насколько (смутно) помню результат проверки -- да.
Comment 4 Sergey V Turchin 2008-03-26 17:12:47 MSK
(In reply to comment #3)
> Насколько (смутно) помню результат проверки -- да.
Я тоже так же помню, что -- нет ;-)
Comment 5 Michael Shigorin 2008-03-26 18:33:58 MSK
(In reply to comment #4)
> > Насколько (смутно) помню результат проверки -- да.
> Я тоже так же помню, что -- нет ;-)
Хорошо, проверю при оказии (следующей сборке/заливке terminal), чтоб уж точно
знать :)

Собсно как описано -- на сейчас без специальных мер (которые по умолчанию нигде
не предпринимаются) не-None для ремонтых дисплеев приводит к неожиданному для
пользователя результату.

Бишь действительно лучше вообще убрать кнопочку, а Root или All сделать тогда,
когда действия будут производиться разумные (если разные -- например, root может
положить терминальный сервер, а вот пользователь -- только свой терминал -- то
надо бы продумать ещё и как это сделать... в общем, дело небыстрое).

Я-то могу это в хуках захачить, но некрасивенько же.
Comment 6 Sergey V Turchin 2008-03-26 18:38:45 MSK
(In reply to comment #5)
> Хорошо, проверю при оказии
Проверь, пожалуйста, если успеешь раньше меня
Comment 7 Michael Shigorin 2008-04-11 21:58:37 MSD
Created attachment 2565 [details]
kdmrc package-to-altsp difference

Хм, прилагаю нашу текущую разницу в kdmrc.

При попытке зашатдаунить с терминала кнопочки "выключить/ребутнуть" есть после
K Menu -> выйти (и отсутствуют после правой_по_десктопу->выйти, но это вроде
где-то ещё трогали).  Попытка выключить приводит к окошку с запросом
("Выключить компьютер"; "Требуется разрешение администратора"; "Пароль: [  
]").

Это поведение обусловлено AllowShutdown=Root в секции [X-*-Core]; сменил на
None (как предлагаю по умолчанию) -- о, именно так и хотел.
Comment 8 Michael Shigorin 2008-04-11 21:59:52 MSD
Created attachment 2566 [details]
kdmrc package-to-nicedefault difference

Вот что вышло от пакетного kdmrc до того, который не смущает пользователя за
_удалённым_ дисплеем предложением потушить систему, на которой он работает.

Так и предлагаю модифицировать kde-settings-kdm.
Comment 9 Sergey V Turchin 2008-04-14 15:23:06 MSD
(In reply to comment #8)
> Так и предлагаю модифицировать kde-settings-kdm.
Не, уж лучше самостоятельно
Comment 10 Michael Shigorin 2008-04-14 15:27:55 MSD
(In reply to comment #9)
> > Так и предлагаю модифицировать kde-settings-kdm.
> Не, уж лучше самостоятельно
Чем лучше?  То, что здесь предлагается -- никак не влияет на локальное
использование.
Comment 11 Michael Shigorin 2008-04-14 15:39:51 MSD
(In reply to comment #8)
> Так и предлагаю модифицировать kde-settings-kdm.
Виноват!  Про XDMCP забыл выкинуть :-)  С ним сами разберёмся.

Нужно только AllowShutdown, их несколько и править скриптом сложнее (и смысла
больше поправить в пакете, в отличие от).
Comment 12 Sergey V Turchin 2008-04-14 15:42:53 MSD
(In reply to comment #10)
> То, что здесь предлагается -- никак не влияет на локальное
> использование.
Да, не доглядел.
AllowShutdown=None сделаю

kde-settings >= 3.5.9