FS#73518 - [electron] 16.0.7-2 makes discord crash when scrolling down

Attached to Project: Community Packages
Opened by Paragoumba (Paragoumba) - Wednesday, 26 January 2022, 17:01 GMT
Last edited by Evangelos Foutras (foutrelis) - Thursday, 27 January 2022, 20:52 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Evangelos Foutras (foutrelis)
Nicola Squartini (tensor5)
Architecture All
Severity High
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 6
Private No

Details

Description:
Discord crashes when scrolling down if electron has been updated to version 16.0.7-2

Additional info:
* happens with version 16.0.7-2
* downgrading to version 16.0.7-1 fixes the bug

Steps to reproduce:
- Update electron to version 16.0.7-2
- Start Discord
- Scroll down in any view
- Discord will crash and restart
This task depends upon

Closed by  Evangelos Foutras (foutrelis)
Thursday, 27 January 2022, 20:52 GMT
Reason for closing:  Fixed
Additional comments about closing:  electron 16.0.7-3, electron13 13.6.8-2
Comment by Cleo (cleo404) - Wednesday, 26 January 2022, 20:06 GMT
Just to clarify, in my experience it works fine with the discord and discord-canary packages in the official repos, as they use discord's custom electron.
The bug only appears when discord is modified to work with system electron. The two aur packages discord_arch_electron, and discord-canary-electron-bin, do this.
discord-canary-electron-bin is waiting on a solution on arch's packaging's end, and discord_arch_electron switched to electron 15 in order to avoid the bug.
Comment by Paragoumba (Paragoumba) - Wednesday, 26 January 2022, 20:14 GMT
Yes I should have been more clear
Comment by Scott K. (Pillgar) - Thursday, 27 January 2022, 00:19 GMT
I'm seeing this bug also. Anything that uses Electron 16.0.7-2 coredumps when scrolling. Pastebin link: https://pastebin.com/LsVRHxVR
Comment by Jouni Rinne (cyberpunkrocker) - Thursday, 27 January 2022, 06:09 GMT
Can confirm. EVERYTHING using electron-16.0.7-2 crashes the same way, rolling back to -1 fixes things.
Comment by Paragoumba (Paragoumba) - Thursday, 27 January 2022, 17:11 GMT
electron13 version 13.6.8-1 suffers from the same issue. Downgrading to the previous version (13.6.7-1) also fixes the crash
Comment by Evangelos Foutras (foutrelis) - Thursday, 27 January 2022, 19:17 GMT
electron 16.0.7-3 should work.[1] I was seeing similar crashes in Chromium which I attributed to -fstack-clash-protection.

@Nicola: The same fix should work for electron13. The curious bit though is why the previous builds didn't exhibit this issue, even though they were built with the same devtools release.

[1] Tested with `electron /usr/share/doc/clang/html/index.html` which crashed with 16.0.7-2 but works now.
Comment by Evangelos Foutras (foutrelis) - Thursday, 27 January 2022, 19:49 GMT
@Nicola: Never mind, I see options=('!lto') was added to electron13 13.6.8-1. So there must be some interaction with ThinLTO and -fstack-clash-protection.

I'll go ahead and rebuild electron13 without -fstack-clash-protection for now.

Loading...