Compiz crash on startup
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Geis |
Fix Released
|
High
|
Daniel d'Andrada | ||
geis (Ubuntu) |
Fix Released
|
Undecided
|
Daniel d'Andrada | ||
Quantal |
Fix Released
|
Undecided
|
Daniel d'Andrada |
Bug Description
Thanks for any help. Ubuntu 12.10. After login Compiz crashes, and when I try to lunch it in terminal it doesn't work and the output is:
per-olav@asus12:~$ compiz
compiz (core) - Info: Loading plugin: core
compiz (core) - Info: Starting plugin: core
[...]
ERROR 2012-11-18 17:24:14 nux.gestures_
Minnesegmentsfeil (core dumped)
[Impact]
In some computers, users are unable to get past the login screen. Compiz crashes right after it's started.
The fix should be backported as it's a simple null pointer check (i.e., low regression risk) and solves the serious problem described above. The crash happened because, in nux, when geis fails to activate a gestures subscription (which is an uncommon case, thus the low number of computers affected), nux destroys the related objects, which includes a pointer to a geis filter. In this situation that geis filter pointer is null and therefore geis_filter_
[Test Case]
A unit test has been included along with the fix.
For users with computers affected by this bug, testing the fix is really simple: after installing the package with the proposed fix, just try to log in and check if you get to the ubuntu desktop successfully.
[Regression Potential]
Only computers affected by this bug will get a different code path executed due to this fix, as they are the only ones that pass null pointers to geis_filter_
Related branches
- Stephen M. Webb (community): Approve
- PS Jenkins bot: Pending (continuous-integration) requested
-
Diff: 79 lines (+43/-2)4 files modifieddebian/changelog (+6/-0)
libgeis/geis_filter.c (+1/-1)
testsuite/geis2/Makefile.am (+2/-1)
testsuite/geis2/gtest_gbe_null_filter.cpp (+34/-0)
Changed in geis: | |
status: | New → Confirmed |
Changed in compiz: | |
status: | New → Confirmed |
Changed in geis: | |
status: | Incomplete → Confirmed |
Changed in geis: | |
importance: | Undecided → Medium |
assignee: | nobody → Daniel d'Andrada (dandrader) |
affects: | geis → nux |
Changed in nux: | |
status: | Confirmed → In Progress |
Changed in geis: | |
assignee: | nobody → Daniel d'Andrada (dandrader) |
importance: | Undecided → High |
status: | New → In Progress |
no longer affects: | nux |
no longer affects: | compiz |
Changed in geis: | |
status: | In Progress → Fix Committed |
Changed in geis (Ubuntu Quantal): | |
assignee: | nobody → Daniel d'Andrada (dandrader) |
status: | New → In Progress |
Changed in geis (Ubuntu): | |
status: | New → In Progress |
assignee: | nobody → Daniel d'Andrada (dandrader) |
description: | updated |
Changed in geis: | |
status: | Fix Committed → Fix Released |
Changed in geis: | |
milestone: | none → 2.2.16 |
Changed in geis: | |
milestone: | 2.2.16 → none |
That nux.gestures_ subscription error message by itself doesn't imply that there as a crash in geis.
A backtrace would be nice.