system/serenity
system/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity synced 2024-09-06 08:56:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

WindowServer: Fix mapping the correct framebuffer size

Browse source 
If we don't support double buffering for a certain type of hardware,
don't try to map with size calculated with (pitch * height * 2), as it
will result in trying to map more memory than is available in the
framebuffer memory range.
This commit is contained in:
Liav A 2022-09-23 15:59:53 +03:00 committed by Linus Groh
parent 76ace3629a
commit 36f2e85823
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
Userland/Services/WindowServer/HardwareScreenBackend.cpp
View file

@ -85,7 +85,11 @@ ErrorOr<void> HardwareScreenBackend::map_framebuffer()
if (rc != 0) {
return Error::from_syscall("graphics_connector_get_head_mode_setting"sv, rc);
}
m_size_in_bytes = mode_setting.horizontal_stride * mode_setting.vertical_active * 2;
if (m_can_set_head_buffer) {
m_size_in_bytes = mode_setting.horizontal_stride * mode_setting.vertical_active * 2;
} else {
m_size_in_bytes = mode_setting.horizontal_stride * mode_setting.vertical_active;
}
m_framebuffer = (Gfx::ARGB32*)TRY(Core::System::mmap(nullptr, m_size_in_bytes, PROT_READ | PROT_WRITE, MAP_SHARED, m_display_connector_fd, 0));
if (m_can_set_head_buffer) {