Bug 40585 - lightdm: use-after-free, потенциально приводящий к прекращению работы функции запуска "предыдущей сессии"
Summary: lightdm: use-after-free, потенциально приводящий к прекращению работы функции...
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: lightdm (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 normal
Assignee: manowar@altlinux.org
QA Contact: qa-sisyphus
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2021-07-27 13:45 MSK by Aleksei Nikiforov
Modified: 2021-08-20 03:33 MSK (History)
3 users (show)

See Also:


Attachments
lightdm.gdb.log (16.76 KB, text/x-log)
2021-07-27 13:45 MSK, Aleksei Nikiforov
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Aleksei Nikiforov 2021-07-27 13:45:13 MSK
Created attachment 9538 [details]
lightdm.gdb.log

При попытке запустить lightdm-kde-greeter, адаптированный под qt5/kde5, при запуске с предыдущей сессией, запуск не работает.

Воспроизведение:
1) Взять K Workstation, обновить до Сизифа и поставить lightdm и lightdm-kde-greeter.
Сейчас lightdm-kde-greeter - work in progress, текущее состояние сейчас можно взять отсюда:
http://git.altlinux.org/people/darktemplar/packages/?p=lightdm-kde-greeter.git;a=shortlog;h=refs/heads/sisyphus
2) Выставить в конфиге lightdm-kde-greeter тему classic:
$ cat /etc/lightdm/lightdm-kde-greeter.conf 
[greeter]
theme-name=classic
3) Запустить lightdm, убедиться что в качестве greeter используется lightdm-kde-greeter или же заставить использовать его.
4) Убедиться что в сессии выбран пункт "Previously Used Session". Он обычно выбран по-умолчанию.
5) Залогиниться в сессию. Может потребоваться сделать это несколько раз одним и тем же пользователем для заполнения и использования файла ~/.dmrc.

Результат:
Пользовательская сессию не запускается.
В логе /var/log/lightdm/lightdm.log подобные записи:
[+1139.38s] DEBUG: Greeter requests default session
[+1148.40s] CRITICAL: g_variant_new_string: assertion 'g_utf8_validate (string, -1, NULL)' failed
[+1148.40s] CRITICAL: g_dbus_connection_call_sync_internal: assertion 'G_IS_DBUS_CONNECTION (connection)' failed
[+1153.94s] DEBUG: Writing /home/test/.dmrc
[+1424.62s] DEBUG: Seat seat0: Failed to find session configuration �\���U
[+1424.62s] DEBUG: Seat seat0: Can't find session '�\���U'
[+2667.24s] DEBUG: Got signal 15 from process 1

Ожидаемый результат:
Запуск предыдущей пользовательской сессии.


Дополнительная информация:
Покопался с gdb и нашёл проблему. Лог gdb, почищенный от большей части ненужного, прикреплён.

Судя по результатам, патчем lightdm-1.30.0-reread-dmrc-alt.patch внесена ошибка типа "use after free": когда прилетает session_name == NULL, session_name берётся из данных, хранящихся в "user", однако затем из-за форсированного вызова  load_dmrc эти данные освобождаются и выделяются/считываются/наполняются снова, однако в session_name остаётся старый невалидный указатель. Дальше всё работает плохо.

В p9 воспроизвести не удалось. Либо везёт, либо что-то ещё блокирует проявление ошибки.

Прошу откатить патч lightdm-1.30.0-reread-dmrc-alt.patch для исправления проблемы.
Comment 1 Aleksei Nikiforov 2021-08-18 11:45:21 MSK
ping
Comment 2 manowar@altlinux.org 2021-08-18 12:06:37 MSK
А на что откатить-то? На предыдущую версию или совсем его убрать?
Comment 3 Aleksei Nikiforov 2021-08-18 12:17:35 MSK
(Ответ для manowar@altlinux.org на комментарий #2)
> А на что откатить-то? На предыдущую версию или совсем его убрать?

Это как Вы решите, просто отрывать патч или как-то его переделывать/переосмыслять. Просто отрывание патча вернёт связанную багу #30329.

Кстати, lightdm-kde-greeter уже вернулся в Сизиф.
Comment 4 Repository Robot 2021-08-20 03:33:10 MSK
lightdm-1.30.0-alt15 -> sisyphus:

 Fri Aug 20 2021 Paul Wolneykien <manowar@altlinux> 1.30.0-alt15
 - Revoke the "reread-dmrc" patch (closes: 40585).
 - ALTBUG 30329 should now be reopened.