Re: [PATCH] drm/syncobj: Fix handle <-> fd ioctls with dirty stack
From: Michel Dänzer
Date: Tue Mar 03 2026 - 12:08:45 EST
On 3/3/26 17:54, Julian Orth wrote:
> On Tue, Mar 3, 2026 at 5:40 PM Maarten Lankhorst
> <maarten.lankhorst@xxxxxxxxxxxxxxx> wrote:
>>
>> There is precedence in the ioctl, the pad member is checked against zero for the same reason.
>
> I don't believe that this is comparable. Developers of code developed
> against an older kernel could look at the kernel and see that the pad
> field was checked against zero. They could not see the same for fields
> that didn't exist at the time.
"Initialize only known fields" isn't a valid approach here. The full struct must be initialized to 0, including any fields added in the future.
--
Earthling Michel Dänzer \ GNOME / Xwayland / Mesa developer
https://redhat.com \ Libre software enthusiast