emvaized / kde-snap-assist

KWin script for KDE Plasma which suggests other window thumbnails on snap
https://store.kde.org/p/1875687
GNU General Public License v3.0
119 stars 7 forks source link

Plasma notifications crash kwin_x11 when Snap Assist is enabled #38

Closed tam1m closed 2 years ago

tam1m commented 2 years ago

I can realiably trigger this by invoking notify-send --expire-time=10 --app-name="$RANDOM" "$RANDOM" -p. The important part is the expire-time on notifications. Non-timed notifications do not trigger the crash at all it seems like. Setting --expire-time to something between 1 and 10 crashes kwin almost all the time. Setting it higher makes the crashes appear to be more random.

Disabling KDE Snap Assist fixes the issue.

To reproduce:

  1. enable KDE Snap Assist
  2. trigger timed notification with notify-send --expire-time=10 --app-name="$RANDOM" "$RANDOM" -p

    Crashlog taken with journalctl --user -u plasma-kwin_x11.service -u plasma-plasmashell.service -f

emvaized commented 2 years ago

What version of the script do you use?

tam1m commented 2 years ago

Version 1.6.1

Edit: I'm a bit confused. Going through Settings > Kwin scripts > About it says 1.6.1. But searching for the script through Discover says the latest script versin is 1.6.2. No updates found though. So I'm not sure if it's actually 1.6.1 or 1.6.2 with the version not updated

Edit 2: Completly un- and reinstalled the script. Now it is 1.6.2 and the problem seems fixed. I guess Discover just didn't want to update properly.

This issue can be closed as version 1.6.2 already fixed this

emvaized commented 2 years ago

Yeah, that's why I asked – because 1.6.2 version includes a fix for this exact kind of issues. This update was published only 8 days ago, maybe it just hasn't gotten to you yet.

I'm glad the issue is resolved for you now