Segfault in app_indicator_set_icon_full [patch attached]
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
libappindicator (Ubuntu) |
Fix Released
|
Medium
|
Marco Trevisan (Treviño) | ||
Bionic |
Fix Released
|
Medium
|
Marco Trevisan (Treviño) |
Bug Description
[ Impact ]
Discord and several other applications using libappindicator are widely reported to have been crashing for several years. See: https:/
[ Test case ]
- Run discord application
- It must not crash in ubuntu (or when indicators are enabled)
[ Regression potential ]
Very low, icons might not appear in some cases, if any.
-----
This is the backtrace:
(gdb) bt full
#0 0x00007fe1d5d2e00e in () at /app/lib/
#1 0x00007fe1f5a6f3c5 in g_closure_invoke () at /lib/libgobject
#2 0x00007fe1f5a813d2 in () at /lib/libgobject
#3 0x00007fe1f5a8a02c in g_signal_
#4 0x00007fe1f5a8a40f in g_signal_emit () at /lib/libgobject
#5 0x00007fe1d5d2ed4f in app_indicator_
#6 0x000000000077851a in ()
#7 0x0000000001de7123 in ()
#8 0x0000000001e4bd4e in ()
#9 0x0000000001e6e34c in ()
#10 0x0000000001e6e668 in ()
#11 0x0000000001e6e9cb in ()
#12 0x0000000001df971a in ()
#13 0x00007fe1f354b1c7 in g_main_
#14 0x00007fe1f354b430 in () at /lib/libglib-
#15 0x00007fe1f354b4dc in g_main_
#16 0x0000000001df9606 in ()
#17 0x0000000001e6e0e7 in ()
#18 0x0000000001e29570 in ()
#19 0x0000000000c37ec8 in ()
#20 0x0000000000c37d15 in ()
#21 0x0000000000c1da7d in ()
#22 0x0000000000a9282e in ()
#23 0x00000000007892d4 in ()
#24 0x00000000007896e0 in ()
#25 0x0000000003b830a3 in main ()
Happens in all versions of libappindicator built from latest sources available on launchpad.
I ran into the issue yesterday when installing Discord for the first time. I have tracked the problem down to libappindicator passing in an extra vararg item to g_signal_emit that the signal's definition in libappindicator was not declaring, causing the crash you see above in gobject's g_signal dispatch machinery.
Patch is attached.
I am presuming this is 'upstream' for libappindicator, whatever that may mean for what appears to be an unmaintained project. If it is not, and since it is an Ubuntu/
Related branches
- Indicator Applet Developers: Pending requested
- Diff: 0 lines
Changed in libappindicator (Ubuntu): | |
assignee: | nobody → Marco Trevisan (Treviño) (3v1n0) |
status: | Confirmed → Fix Committed |
description: | updated |
Changed in libappindicator (Ubuntu Bionic): | |
assignee: | nobody → Marco Trevisan (Treviño) (3v1n0) |
tags: | removed: removal-candidate verification-needed |
affects: | libappindicator (Arch Linux) → ubuntu-translations |
no longer affects: | ubuntu-translations |
Changed in libappindicator (Ubuntu): | |
importance: | Undecided → Medium |
Changed in libappindicator (Ubuntu Bionic): | |
importance: | Undecided → Medium |
The attachment "Stop passing in undeclared boolean vararg" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.
[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]