[BNR] Compiz crash in movementWindowOnScreen (caused by fix 755842)
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Compiz |
Fix Released
|
High
|
Sam Spilsbury | ||
Compiz Main Plugins |
Fix Committed
|
High
|
Sam Spilsbury | ||
compiz (Ubuntu) |
Fix Released
|
High
|
Unassigned | ||
Precise |
Won't Fix
|
Undecided
|
Unassigned | ||
compiz-plugins-main (Ubuntu) |
In Progress
|
High
|
Unassigned | ||
Precise |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
[Impact]
A crash after fix to bug #755842.
[Test case]
Use super+W rapidly a few times.
[Regression Potential]
Should be low, correct usage of the vector variable. Test wall plugin's offset movement functionality in general.
TESTCASE:
ctest -D ExperimentalMem
(this option was only added on 20 June 2012)
ORIGINAL DESCRIPTION:
Running compiz 1:0.9.8+
valgrind says:
==3648== Invalid read of size 2
==3648== at 0x806092A: compiz:
==3648== by 0x44BCEC7: ???
==3648== Address 0x44bd026 is 14 bytes inside a block of size 20 free'd
==3648== at 0x402ACFC: operator delete(void*) (in /usr/lib/
==3648== by 0x8060965: compiz:
==3648== by 0x44BCEC7: ???
The classic case of iterating a temporary vector that goes out of scope while iterating. Annoying.
Related branches
- Daniel van Vugt: Pending requested
- Diff: 0 lines
- Daniel van Vugt: Approve
-
Diff: 25 lines (+5/-3)1 file modifiedplugins/wall/src/offset_movement/src/offset-movement.cpp (+5/-3)
Changed in compiz: | |
milestone: | none → 0.9.8.0 |
assignee: | nobody → Sam Spilsbury (smspillaz) |
importance: | Undecided → High |
status: | New → In Progress |
description: | updated |
Changed in compiz-core: | |
importance: | Undecided → High |
milestone: | none → 0.9.7.10 |
description: | updated |
Changed in compiz: | |
status: | In Progress → Fix Committed |
affects: | compiz (Ubuntu) → compiz-plugins-main (Ubuntu) |
no longer affects: | compiz-plugins-main (Ubuntu) |
Changed in compiz: | |
status: | Fix Committed → Fix Released |
description: | updated |
Changed in compiz-plugins-main (Ubuntu): | |
status: | Triaged → In Progress |
Note: This bug was never in precise. But the fix does need to be backported to lp:compiz-plugins-main before maintenance release 0.9.7.4 of that project.