]> www.infradead.org Git - users/jedix/linux-maple.git/commit
drm/vboxvideo: Replace fake VLA at end of vbva_mouse_pointer_shape with real VLA
authorHans de Goede <hdegoede@redhat.com>
Tue, 27 Aug 2024 10:45:23 +0000 (12:45 +0200)
committerHans de Goede <hdegoede@redhat.com>
Sat, 14 Sep 2024 11:32:15 +0000 (13:32 +0200)
commitd92b90f9a54d9300a6e883258e79f36dab53bfae
treef9ff853335015daa13641ebcc74d71d2b62eb9f5
parent4a33aa34e4ac2c2bd746a305d4b39fb60dedc091
drm/vboxvideo: Replace fake VLA at end of vbva_mouse_pointer_shape with real VLA

Replace the fake VLA at end of the vbva_mouse_pointer_shape shape with
a real VLA to fix a "memcpy: detected field-spanning write error" warning:

[   13.319813] memcpy: detected field-spanning write (size 16896) of single field "p->data" at drivers/gpu/drm/vboxvideo/hgsmi_base.c:154 (size 4)
[   13.319841] WARNING: CPU: 0 PID: 1105 at drivers/gpu/drm/vboxvideo/hgsmi_base.c:154 hgsmi_update_pointer_shape+0x192/0x1c0 [vboxvideo]
[   13.320038] Call Trace:
[   13.320173]  hgsmi_update_pointer_shape [vboxvideo]
[   13.320184]  vbox_cursor_atomic_update [vboxvideo]

Note as mentioned in the added comment it seems the original length
calculation for the allocated and send hgsmi buffer is 4 bytes too large.
Changing this is not the goal of this patch, so this behavior is kept.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240827104523.17442-1-hdegoede@redhat.com
drivers/gpu/drm/vboxvideo/hgsmi_base.c
drivers/gpu/drm/vboxvideo/vboxvideo.h