miral-kiosk crashes on ldiv0 only on raspi3 with webkit
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Mir |
Fix Released
|
Critical
|
Alan Griffiths | ||
| MirAL |
Fix Committed
|
Critical
|
Alan Griffiths | ||
Bug Description
miral-kiosk crashing specificity with a mir-kiosk-apps fancybrowser (or any webkit example)
How to reproduce:
1) Install a ubuntu core image for the raspi3 http://
2) snap install mir-libs --edge
3) snap install mir-kiosk --edge
4) install this snap https:/
5) snap install mir-kiosk-
6) with the mir-kiosk running and the clocks kiosk-app running type:
snap set mir-kiosk-apps app=fancybrowser
Stacktrace:
http://
miral logging:
http://
qt logging:
http://
It seems for unknown reasons in miral::
From the logs them selfs I dont see *any* crazy numbers that would cause the width to become negative.
A fix is up, and a test that confirms that negative width bit but unsure how it got to be in that state.
https:/
Related branches
- Mir CI Bot: Needs Fixing (continuous-integration)
- Alan Griffiths: Needs Information
- Alberto Aguirre (community): Approve
-
Diff: 116 lines (+66/-4)3 files modifiedsrc/miral/window_info.cpp (+4/-4)
tests/miral/CMakeLists.txt (+1/-0)
tests/miral/window_info.cpp (+61/-0)
- Mir CI Bot: Needs Fixing (continuous-integration)
- Mir development team: Pending requested
-
Diff: 15 lines (+3/-2)1 file modifiedmiral/window_info.cpp (+3/-2)
- Mir CI Bot: Needs Fixing (continuous-integration)
- Gerry Boland (community): Approve
-
Diff: 316 lines (+160/-23)7 files modifieddebian/changelog (+2/-0)
src/miral/CMakeLists.txt (+1/-0)
src/miral/window_info.cpp (+49/-15)
src/miral/window_info_defaults.h (+36/-0)
src/miral/window_management_trace.cpp (+10/-8)
tests/miral/CMakeLists.txt (+1/-0)
tests/miral/window_info.cpp (+61/-0)
- Mir CI Bot: Approve (continuous-integration)
- Gerry Boland (community): Approve
- Brandon Schaefer (community): Approve
-
Diff: 350 lines (+174/-39)7 files modifiedmiral/CMakeLists.txt (+1/-0)
miral/set_window_management_policy.cpp (+16/-16)
miral/window_info.cpp (+49/-15)
miral/window_info_defaults.h (+36/-0)
miral/window_management_trace.cpp (+10/-8)
test/CMakeLists.txt (+1/-0)
test/window_info.cpp (+61/-0)
- 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)
| description: | updated |
| description: | updated |
| description: | updated |
| Changed in miral: | |
| status: | New → In Progress |
| importance: | Undecided → Critical |
| assignee: | nobody → Brandon Schaefer (brandontschaefer) |
| Changed in mir: | |
| status: | New → In Progress |
| importance: | Undecided → Critical |
| assignee: | nobody → Alan Griffiths (alan-griffiths) |
| milestone: | none → 1.0.0 |
| Changed in miral: | |
| assignee: | Brandon Schaefer (brandontschaefer) → nobody |
| assignee: | nobody → Alan Griffiths (alan-griffiths) |
| Changed in mir: | |
| status: | In Progress → Fix Committed |
| Mir CI Bot (mir-ci-bot) wrote | #1 |
| Changed in miral: | |
| status: | In Progress → Fix Committed |
| Changed in mir: | |
| status: | Fix Committed → Fix Released |
Fix committed into lp:miral at revision 580, scheduled for release in miral, milestone Unknown