diff options
| author | Andrea Azzarone <azzaronea@gmail.com> | 2016-04-05 19:16:29 +0200 |
|---|---|---|
| committer | Andrea Azzarone <azzaronea@gmail.com> | 2016-04-05 19:16:29 +0200 |
| commit | fd0435071434f213e4508e334f05bc591947f9a1 (patch) | |
| tree | 392bc44ce4149b623a464674647492a4e98b6dab /lockscreen | |
| parent | fb16f85634d8ecf260c128e039e108396cdbd9f9 (diff) | |
Add comment.
(bzr r4095.2.3)
Diffstat (limited to 'lockscreen')
| -rw-r--r-- | lockscreen/LockScreenController.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lockscreen/LockScreenController.cpp b/lockscreen/LockScreenController.cpp index 98bdd90a0..0caf8ba17 100644 --- a/lockscreen/LockScreenController.cpp +++ b/lockscreen/LockScreenController.cpp @@ -417,10 +417,11 @@ void Controller::OnScreenSaverActivationRequest(bool activate) { if (Settings::Instance().use_legacy()) { + // SetActive(FALSE) will unlock the screen. This used to cause security issues (see lp:1552537). if (activate) { auto proxy = std::make_shared<glib::DBusProxy>("org.gnome.ScreenSaver", "/org/gnome/ScreenSaver", "org.gnome.ScreenSaver"); - proxy->CallBegin("SetActive", g_variant_new("(b)", activate != FALSE), [proxy] (GVariant*, glib::Error const&) {}); + proxy->CallBegin("SetActive", g_variant_new("(b)", TRUE), [proxy] (GVariant*, glib::Error const&) {}); } return; } |