Bug 376755 - Screen locker is delayed, exposing data
Summary: Screen locker is delayed, exposing data
Status: RESOLVED DUPLICATE of bug 366402
Alias: None
Product: Powerdevil
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Development Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-02-21 09:08 UTC by Dan Dart
Modified: 2017-02-22 10:34 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
attachment-6492-0.html (28 bytes, text/html)
2017-02-21 10:19 UTC, Martin Flöser
Details
kde info without compositing (3.10 KB, text/x-log)
2017-02-21 22:08 UTC, Dan Dart
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dan Dart 2017-02-21 09:08:28 UTC
When waking from resume, the screen locker is delayed by sometimes up to a minute, at which point any videos resume *in the foreground* (defeating the point of screen locker as a privacy aid), and will not let me interact with anything until the password prompt finally appears, at which point I can unlock and interact.

Should it not instantly appear and block anything from view before unlock?

Cheers
Comment 1 Martin Flöser 2017-02-21 10:19:08 UTC
Created attachment 104142 [details]
attachment-6492-0.html

Which version are you using?
Comment 2 Dan Dart 2017-02-21 10:24:18 UTC
I'm on 5.8.5-0ubuntu1~ubuntu16.10~ppa1 - from https://launchpad.net/~kubuntu-ppa/+archive/ubuntu/backports?field.series_filter=yakkety

(not official ubuntu packages apparently, backports)
Cheers
Comment 3 Martin Flöser 2017-02-21 16:02:02 UTC
How do you suspend the system? Can you please provide the output of:
qdbus org.kde.KWin /KWin supportInformation

Please note that the screenlocker does ensure that the screen is locked prior to suspending. We have cases where incorrectly one frame is shown by the GPU driver, but a case like yours has not been described in years and I spent days ensuring that this cannot happen.
Comment 4 Dan Dart 2017-02-21 16:04:46 UTC
I just close the lid and the laptop suspends by ACPI.
It resumes and I can see all the data as above.

Here's the support info:
https://paste.kde.org/plxnvff4o

It has also happened before compositing was disabled.
Comment 5 Martin Flöser 2017-02-21 19:34:03 UTC
Could you also provide the support information with compositing. That has the interesting part for me. Please also just paste here instead of paste.kde.org as that expires.

> I just close the lid and the laptop suspends by ACPI.

You are using powerdevil for that? Could you try manually suspend without closing the lid?
Comment 6 Dan Dart 2017-02-21 22:07:52 UTC
OK - I tried manually suspending - this appeared to work just fine.
So am I to understand that this isn't a kscreenlocker bug - but a bug with the way suspends are (or are not) handled?
Comment 7 Dan Dart 2017-02-21 22:08:25 UTC
Created attachment 104157 [details]
kde info without compositing
Comment 8 Martin Flöser 2017-02-22 06:08:45 UTC
> So am I to understand that this isn't a kscreenlocker bug - but a bug with the way suspends are (or are not) handled?

Yes, looks like it. I'm reassigning to powerdevil
Comment 9 Kai Uwe Broulik 2017-02-22 07:39:54 UTC
From what I can tell, closing the lid just simulates a press on the suspend button, so in theory there shouldn't be a difference between manually suspending and closing the lid.

What version of systemd and logind are you using? I recall there having been a behavior change in some versions where it would ignore the lid inhibition to let us handle that event and instead would just suspend unconditionally, giving the screen locker no chance to start in time.
Comment 10 Dan Dart 2017-02-22 09:52:58 UTC
Sounds like something reasonable.

I've got version 231 of systemd (not sure how to check for logind - same package I presume?
Comment 11 Kai Uwe Broulik 2017-02-22 10:07:06 UTC
I think that's exactly the version where it broke, sorry. :/

See Bug 366402 for a full explanation (and a workaround), closing this as duplicate then. Thanks for your feedback!

*** This bug has been marked as a duplicate of bug 366402 ***
Comment 12 Dan Dart 2017-02-22 10:34:10 UTC
Thanks for confirming et al