VS Code crashes Vala Appmenu
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
vala-panel-appmenu (Ubuntu) |
Triaged
|
Medium
|
fossfreedom | ||
Focal |
Fix Released
|
Medium
|
fossfreedom | ||
Groovy |
Fix Released
|
Medium
|
fossfreedom |
Bug Description
[Impact]
The appmenu provides a in-panel global menu implementation for a number of desktop environments.
With QT apps such as VSCode, performing certain functions makes the appmenu produce a segmentation fault. Under Budgie, the panel restarts - and does cause the whole user session to be restarted. Under XFCE a dialog is displayed indicating that the appmenu has crashed.
This upload backports a series of upstream patches designated by the maintainer as a resolution of this issue (https:/
This occurs with the v0.7.3 series of appmenu uploaded to focal and groovy. v0.7.6 that is included with hirsute already contains the fixes.
In addition whilst developing this patch it was found that the package itself FTBFS. A patch from upstream is included to resolve this issue.
[Test Plan]
* If using the snap version of vscode, start vscode and observe the menu in the panel. Click on the panel menu option. If using Budgie, observe that the panel restarts i.e. crashes. If using XFCE observe a dialog displayed indicating a crash.
* If using the deb version of vscode, start vscode and observe the menu in the panel. Click on the File menu and attempt to open a folder. If using Budgie, bbserve that the panel restarts i.e. crashes. If using XFCE observe a dialog displayed indicating a crash.
* Next enable the proposed repository and install the following package:
For budgie-desktop budgie-
For mate-desktop mate-applet-appmenu
For xfce-desktop xfce4-appmenu-
* Logout and login
* Repeat the vscode tests above - note this time that the panel crash actions do not occur.
* Next confirm that no additional panel instabilities occur with the following application types
Libreoffice
Gtk based game such as Aislerot (solitaire) that do have menus
CSD based apps such as GNOME Calculator that dont have global menu support
Try a broad range of apps found on your desktop menu.
[Where problems could occur]
* Any potential issues with this upload is strictly limited to the package itself - there will be no regression in any other components since the package (per desktop) is self contained.
* As to the package itself - I suppose the worse case scenario is that not all the patches have been identified and cherry-picked and potentially there could be a further regression causing panel instability.
* To mitigate this I have carefully reviewed the commits released since 0.7.3 was released by upstream and I haven't identified any commits specifically in this area of code that has been patched.
* The worse case scenario is that the global menu for a users desktop will have to be removed by the user if any additional regressions are identified over and beyond the issues being fixed.
* Additional mitigation of the risks are the additional regression tests added to the test plan to cover a broader range to application types beyond the Qt apps crash issue being fixed.
[Other Info]
* N/A
----
$lsb_release -rd
Description: Ubuntu 20.04.2 LTS
Release: 20.04
Opening VS Code, particularly folders, causes the Vala-panel-appmenu to have segment fault on xfce4 and in the case of Budgie ends the entire user session as the budgie-panel cannot handle it. Backporting commits from 0.7.5 into the current release appears to fix the issue, I believe fossfreedom also backported another commit to aid in building the source properly.
The crash log of Budgie-panel caused by vala-panel-appmenu has been attached.
https:/
More information can be found here
https:/
https:/
https:/
https:/
https:/
description: | updated |
summary: |
- VS Code crashes Vala Appmenu + VS Code crashes Vala Appmenu - patch included |
description: | updated |
Changed in vala-panel-appmenu (Ubuntu): | |
status: | Confirmed → Triaged |
summary: |
- VS Code crashes Vala Appmenu - patch included + VS Code crashes Vala Appmenu |
description: | updated |
description: | updated |
description: | updated |
tags: | added: patch |
I'll tidy up the patch and will submit it for consideration for the stable release series.
Note - a patch will be needed for both 20.04 and 20.10 since vala-panel-appmenu is similar (but not quite the same)
Hirsute is not impacted since the upstream fixes are in the hirsute release.