<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>450331</bug_id>
          
          <creation_ts>2022-02-15 18:58:18 +0000</creation_ts>
          <short_desc>when overview is mapped a modifier only shortcut, it works even at lock screen, thus the user&apos;s current windows and their contents are exposed.</short_desc>
          <delta_ts>2022-02-17 15:01:35 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Plasma</classification>
          <product>kwin</product>
          <component>effects-overview</component>
          <version>unspecified</version>
          <rep_platform>Other</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>VHI</priority>
          <bug_severity>critical</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>partialtemplate</reporter>
          <assigned_to name="KWin default assignee">kwin-bugs-null</assigned_to>
          <cc>kde</cc>
    
    <cc>nate</cc>
          
          <cf_commitlink>https://invent.kde.org/plasma/kwin/commit/39153cf77aac120476402b21c9fdd357ec1d40ce</cf_commitlink>
          <cf_versionfixedin>5.24.2</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      <flag name="Wayland"
          id="4605"
          type_id="31"
          status="-"
          setter="nate"
    />
    <flag name="X11"
          id="4606"
          type_id="32"
          status="+"
          setter="nate"
    />

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2105013</commentid>
    <comment_count>0</comment_count>
    <who name="">partialtemplate</who>
    <bug_when>2022-02-15 18:58:18 +0000</bug_when>
    <thetext>SUMMARY
when overview is mapped a modifier only shortcut, it works even at the lock screen, thus the user&apos;s current windows and their contents are exposed.

STEPS TO REPRODUCE
1. enable the overview effect
2. lock the screen(default : Meta+L) 
3. activate overview effect via a modifier-only-shortcut

OBSERVED RESULT
overview effect activates, the contents of all windows in the current desktop session are shown.
overview itself is unresponsive to user input.
In addition, the CPU seems to overheat.

EXPECTED RESULT
overview effect should not activate when the user is unauthenticated.

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma:  
(available in About System)
KDE Plasma Version: 5.24.1
KDE Frameworks Version: 5.91.0
Qt Version: 5.15.2

ADDITIONAL INFORMATION
in kwinrc:
[ModifierOnlyShortcuts]
Alt=
Control=
Meta=org.kde.kglobalaccel,/component/kwin,,invokeShortcut,Overview
Shift=</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2105565</commentid>
    <comment_count>1</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2022-02-16 22:57:49 +0000</bug_when>
    <thetext>Cannot reproduce on Wayland, trying on X11...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2105569</commentid>
    <comment_count>2</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2022-02-16 23:15:22 +0000</bug_when>
    <thetext>Can reproduce on X11. Raising priority and severity due to the security implications.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2105574</commentid>
    <comment_count>3</comment_count>
    <who name="David Edmundson">kde</who>
    <bug_when>2022-02-16 23:44:22 +0000</bug_when>
    <thetext>We don&apos;t expose that in a GUI option anywhere.
Still worth fixing, but not worth being too worried about it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2105576</commentid>
    <comment_count>4</comment_count>
    <who name="Bug Janitor Service">bug-janitor</who>
    <bug_when>2022-02-16 23:56:25 +0000</bug_when>
    <thetext>A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/2034</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2105738</commentid>
    <comment_count>5</comment_count>
    <who name="Vlad Zahorodnii">vlad.zahorodnii</who>
    <bug_when>2022-02-17 13:42:45 +0000</bug_when>
    <thetext>Git commit aab395f07bcfeca487b7736ddc10902d8510768c by Vlad Zahorodnii, on behalf of David Edmundson.
Committed on 17/02/2022 at 13:01.
Pushed by vladz into branch &apos;master&apos;.

Check lockscreen status for fullscreen effects

Whilst global shortcuts are blocked by grabbing the keyboard, user set
up manual scripts can still invoke a global action.

Given we already have code to deactivate when locking it makes sense to
also prevent activation.

M  +3    -0    src/effects/desktopgrid/desktopgrid.cpp
M  +3    -0    src/effects/overview/overvieweffect.cpp
M  +3    -0    src/effects/presentwindows/presentwindows.cpp

https://invent.kde.org/plasma/kwin/commit/aab395f07bcfeca487b7736ddc10902d8510768c</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2105743</commentid>
    <comment_count>6</comment_count>
    <who name="Vlad Zahorodnii">vlad.zahorodnii</who>
    <bug_when>2022-02-17 13:51:36 +0000</bug_when>
    <thetext>Git commit 39153cf77aac120476402b21c9fdd357ec1d40ce by Vlad Zahorodnii, on behalf of David Edmundson.
Committed on 17/02/2022 at 13:51.
Pushed by vladz into branch &apos;Plasma/5.24&apos;.

Check lockscreen status for fullscreen effects

Whilst global shortcuts are blocked by grabbing the keyboard, user set
up manual scripts can still invoke a global action.

Given we already have code to deactivate when locking it makes sense to
also prevent activation.


(cherry picked from commit aab395f07bcfeca487b7736ddc10902d8510768c)

M  +3    -0    src/effects/desktopgrid/desktopgrid.cpp
M  +3    -0    src/effects/overview/overvieweffect.cpp
M  +3    -0    src/effects/presentwindows/presentwindows.cpp

https://invent.kde.org/plasma/kwin/commit/39153cf77aac120476402b21c9fdd357ec1d40ce</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>