diff --git a/Kernel/Graphics/VirtIOGPU/GPU3DDevice.cpp b/Kernel/Graphics/VirtIOGPU/GPU3DDevice.cpp index 03673762376..c7b8e34cd92 100644 --- a/Kernel/Graphics/VirtIOGPU/GPU3DDevice.cpp +++ b/Kernel/Graphics/VirtIOGPU/GPU3DDevice.cpp @@ -114,7 +114,8 @@ ErrorOr GPU3DDevice::ioctl(OpenFileDescription& description, unsigned requ .array_size = spec.array_size, .last_level = spec.last_level, .nr_samples = spec.nr_samples, - .flags = spec.flags + .flags = spec.flags, + .padding = 0, }; MutexLocker locker(m_graphics_adapter.operation_lock()); auto resource_id = m_graphics_adapter.create_3d_resource(resource_spec).value(); diff --git a/Kernel/Graphics/VirtIOGPU/GraphicsAdapter.cpp b/Kernel/Graphics/VirtIOGPU/GraphicsAdapter.cpp index 2ec8b461e3d..23376d31f53 100644 --- a/Kernel/Graphics/VirtIOGPU/GraphicsAdapter.cpp +++ b/Kernel/Graphics/VirtIOGPU/GraphicsAdapter.cpp @@ -284,7 +284,10 @@ ResourceID GraphicsAdapter::create_3d_resource(Protocol::Resource3DSpecification request.resource_id = resource_id.value(); // TODO: Abstract this out a bit more u32* start_of_copied_fields = &request.target; - memcpy(start_of_copied_fields, &resource_3d_specification, sizeof(Protocol::Resource3DSpecification)); + + // Validate that the sub copy from the resource_3d_specification to the offset of the request fits. + static_assert((sizeof(request) - offsetof(Protocol::ResourceCreate3D, target) == sizeof(resource_3d_specification))); + memcpy(start_of_copied_fields, &resource_3d_specification, sizeof(resource_3d_specification)); synchronous_virtio_gpu_command(start_of_scratch_space(), sizeof(request), sizeof(response)); diff --git a/Kernel/Graphics/VirtIOGPU/Protocol.h b/Kernel/Graphics/VirtIOGPU/Protocol.h index e8c3483185c..374fab657c3 100644 --- a/Kernel/Graphics/VirtIOGPU/Protocol.h +++ b/Kernel/Graphics/VirtIOGPU/Protocol.h @@ -328,6 +328,7 @@ struct Resource3DSpecification { u32 last_level; u32 nr_samples; u32 flags; + u32 padding; }; }