[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAH2r5muVvaOii15Kg5e0zWzhKdDy5fVh=GjtM=kZ=fyapKxtyQ@mail.gmail.com>
Date: Thu, 8 Aug 2024 22:50:13 -0500
From: Steve French <smfrench@...il.com>
To: Namjae Jeon <linkinjeon@...nel.org>
Cc: "Gustavo A. R. Silva" <gustavoars@...nel.org>, Steve French <sfrench@...ba.org>,
Sergey Senozhatsky <senozhatsky@...omium.org>, Tom Talpey <tom@...pey.com>,
Paulo Alcantara <pc@...guebit.com>, Ronnie Sahlberg <ronniesahlberg@...il.com>,
Shyam Prasad N <sprasad@...rosoft.com>, Bharath SM <bharathsm@...rosoft.com>,
linux-cifs@...r.kernel.org, samba-technical@...ts.samba.org,
linux-kernel@...r.kernel.org, linux-hardening@...r.kernel.org
Subject: Re: [PATCH][next] smb: smb2pdu.h: Use static_assert() to check struct sizes
merged into cifs-2.6.git for-next
On Thu, Aug 8, 2024 at 9:51 PM Namjae Jeon <linkinjeon@...nel.org> wrote:
>
> 2024년 8월 9일 (금) 오전 7:04, Gustavo A. R. Silva <gustavoars@...nel.org>님이 작성:
> >
> > Commit 9f9bef9bc5c6 ("smb: smb2pdu.h: Avoid -Wflex-array-member-not-at-end
> > warnings") introduced tagged `struct create_context_hdr`. We want to
> > ensure that when new members need to be added to the flexible structure,
> > they are always included within this tagged struct.
> >
> > So, we use `static_assert()` to ensure that the memory layout for
> > both the flexible structure and the tagged struct is the same after
> > any changes.
> >
> > Signed-off-by: Gustavo A. R. Silva <gustavoars@...nel.org>
> Acked-by: Namjae Jeon <linkinjeon@...nel.org>
>
> Thanks!
>
--
Thanks,
Steve
Powered by blists - more mailing lists