Mir server crashes with - std::exception::what: Attempt to remove an overlay which was never added or which has been previously removed
Bug #1483779 reported by
Kevin DuBois
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Mir |
Fix Released
|
High
|
Kevin DuBois | ||
| mir (Ubuntu) |
Fix Released
|
High
|
Unassigned | ||
Bug Description
During mm unplug/plug scenarios with USC in silo0:
ERROR: /home/kdub/
Dynamic exception type: N5boost16except
std::exception:
Which throws through thecompositor thread and takes down usc with a termination.
Related branches
lp:~kdub/mir/fix-1483779-silo0
- Daniel van Vugt: Abstain
- Mir development team: Pending requested
-
Diff: 52 lines (+18/-2)2 files modifiedsrc/server/graphics/software_cursor.cpp (+3/-2)
tests/unit-tests/graphics/test_software_cursor.cpp (+15/-0)
lp:~kdub/mir/fix-1483779
- Alan Griffiths: Approve
- PS Jenkins bot (community): Approve (continuous-integration)
- Cemil Azizoglu (community): Approve
-
Diff: 55 lines (+21/-2)2 files modifiedsrc/server/graphics/software_cursor.cpp (+3/-2)
tests/unit-tests/graphics/test_software_cursor.cpp (+18/-0)
lp:~mir-team/mir/silo0
- Mir development team: Pending requested
-
Diff: 785 lines (+300/-34) (has conflicts)33 files modified3rd_party/android-deps/android/input.h (+1/-1)
3rd_party/android-input/android/frameworks/base/services/input/InputReader.cpp (+3/-0)
3rd_party/android-input/android/frameworks/base/services/input/InputTransport.cpp (+3/-2)
cross-compile-chroot.sh (+2/-0)
debian/changelog (+9/-0)
include/client/mir/events/event_builders.h (+8/-0)
include/client/mir_toolkit/events/input/pointer_event.h (+6/-1)
include/test/mir_test/event_matchers.h (+14/-0)
src/client/event_printer.cpp (+2/-0)
src/client/events/event_builders.cpp (+15/-2)
src/client/input/android/android_input_lexicon.cpp (+3/-1)
src/client/input/input_event.cpp (+5/-1)
src/include/common/mir/events/event_private.h (+2/-0)
src/platforms/android/server/display.cpp (+5/-3)
src/platforms/android/server/display.h (+1/-0)
src/platforms/android/server/display_buffer.cpp (+3/-0)
src/platforms/android/server/display_group.cpp (+25/-5)
src/platforms/android/server/display_group.h (+2/-0)
src/platforms/android/server/real_hwc_wrapper.cpp (+1/-1)
src/server/compositor/multi_threaded_compositor.cpp (+2/-2)
src/server/graphics/software_cursor.cpp (+3/-2)
src/server/input/android/input_sender.cpp (+2/-0)
src/server/input/android/input_translator.cpp (+11/-7)
src/server/input/surface_input_dispatcher.cpp (+3/-2)
tests/acceptance-tests/test_client_input.cpp (+34/-0)
tests/mir_test_framework/fake_input_device_impl.cpp (+3/-1)
tests/unit-tests/compositor/test_multi_threaded_compositor.cpp (+37/-0)
tests/unit-tests/graphics/android/test_display.cpp (+45/-0)
tests/unit-tests/graphics/android/test_display_group.cpp (+26/-0)
tests/unit-tests/graphics/test_software_cursor.cpp (+15/-0)
tests/unit-tests/input/android/test_android_input_sender.cpp (+1/-1)
tests/unit-tests/input/android/test_input_translator.cpp (+7/-1)
tests/unit-tests/input/test_input_event.cpp (+1/-1)
lp:mir
- PS Jenkins bot (community): Approve (continuous-integration)
- Alan Griffiths: Approve
- Kevin DuBois (community): Approve
-
Diff: 1373 lines (+579/-161)23 files modified3rd_party/android-input/android/frameworks/base/services/input/MirLog.cpp (+157/-5)
include/test/mir_test_framework/display_server_test_fixture.h (+2/-0)
include/test/mir_test_framework/stub_client_connection_configuration.h (+35/-0)
include/test/mir_test_framework/testing_client_configuration.h (+43/-0)
include/test/mir_test_framework/testing_process_manager.h (+7/-8)
src/client/mir_client_library.cpp (+39/-17)
src/server/graphics/android/hwc_common_device.cpp (+15/-13)
src/server/options/program_option.cpp (+2/-1)
tests/acceptance-tests/test_client_authorization.cpp (+7/-0)
tests/acceptance-tests/test_client_focus_notification.cpp (+2/-9)
tests/acceptance-tests/test_client_input.cpp (+8/-15)
tests/acceptance-tests/test_client_library.cpp (+30/-17)
tests/acceptance-tests/test_focus_selection.cpp (+2/-9)
tests/acceptance-tests/test_server_shutdown.cpp (+2/-9)
tests/acceptance-tests/test_shell_control_of_surface_configuration.cpp (+2/-9)
tests/acceptance-tests/test_surfaceloop.cpp (+3/-10)
tests/acceptance-tests/test_surfaces_with_output_id.cpp (+2/-9)
tests/mir_test_framework/CMakeLists.txt (+1/-0)
tests/mir_test_framework/display_server_test_fixture.cpp (+9/-6)
tests/mir_test_framework/testing_client_options.cpp (+126/-0)
tests/mir_test_framework/testing_process_manager.cpp (+41/-1)
tests/unit-tests/graphics/android/test_hwc_device.cpp (+25/-1)
tests/unit-tests/logging/test_legacy_input_report.cpp (+19/-22)
Revision history for this message
| Daniel van Vugt (vanvugt) wrote | #1 |
| Changed in mir: | |
| milestone: | none → 0.16.0 |
| milestone: | 0.16.0 → none |
Revision history for this message
| Daniel van Vugt (vanvugt) wrote | #2 |
| Changed in mir: | |
| milestone: | none → 0.16.0 |
| Changed in mir: | |
| status: | In Progress → Fix Committed |
| summary: |
- [silo0] exception thrown in cursor management code around monitor - removals/additions + Mir server crashes with - std::exception::what: Attempt to remove an + overlay which was never added or which has been previously removed |
| Changed in mir: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Daniel van Vugt (vanvugt) wrote | #3 |
| Changed in mir (Ubuntu): | |
| importance: | Undecided → High |
| status: | New → Fix Released |
To post a comment you must log in.
Not sure about the target milestone here if kdub is working just on the silo0 branch.