From a2819372d9038f11c53f7999e7bef010f4761aa2 Mon Sep 17 00:00:00 2001
From: CodeAnarchist <144830359+CodeAnarchist@users.noreply.github.com>
Date: Mon, 7 Jul 2025 23:19:23 +0200
Subject: [PATCH] fix VirtioGPUDriver::mark_dirty

added bounds checks to ensure the dirty rectangle doesn't exceed the screen size.
prevents errors from virtio gpu like "transfer bounds outside resource bounds"
---
 .../graph/drivers/virtio_gpu_pci/virtio_gpu_pci.cpp | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/kernel/graph/drivers/virtio_gpu_pci/virtio_gpu_pci.cpp b/kernel/graph/drivers/virtio_gpu_pci/virtio_gpu_pci.cpp
index 90478c47..d75e7fba 100644
--- a/kernel/graph/drivers/virtio_gpu_pci/virtio_gpu_pci.cpp
+++ b/kernel/graph/drivers/virtio_gpu_pci/virtio_gpu_pci.cpp
@@ -408,6 +408,18 @@ int VirtioGPUDriver::try_merge(gpu_rect* a, gpu_rect* b) {
 }
 
 void VirtioGPUDriver::mark_dirty(uint32_t x, uint32_t y, uint32_t w, uint32_t h) {
+    if (x >= screen_size.width || y >= screen_size.height)
+        return;
+
+    if (x + w > screen_size.width)
+        w = screen_size.width - x;
+
+    if (y + h > screen_size.height)
+        h = screen_size.height - y;
+
+    if (w == 0 || h == 0)
+        return;
+
     gpu_rect new_rect = { x, y, w, h };
 
     for (uint32_t i = 0; i < dirty_count; i++) {
@@ -421,6 +433,7 @@ void VirtioGPUDriver::mark_dirty(uint32_t x, uint32_t y, uint32_t w, uint32_t h)
         full_redraw = true;
 }
 
+
 void VirtioGPUDriver::clear(uint32_t color) {
     fb_clear((uint32_t*)framebuffer, screen_size.width, screen_size.height, color);
     mark_dirty(0,0,screen_size.width,screen_size.height);