Skip to content

Commit 5b4f3af

Browse files
GustavoARSilvaSteve French
authored andcommitted
smb: smb2pdu.h: Use static_assert() to check struct sizes
Commit 9f9bef9 ("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. Acked-by: Namjae Jeon <linkinjeon@kernel.org> Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org> Signed-off-by: Steve French <stfrench@microsoft.com>
1 parent 836bb32 commit 5b4f3af

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

fs/smb/common/smb2pdu.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1216,6 +1216,8 @@ struct create_context {
12161216
);
12171217
__u8 Buffer[];
12181218
} __packed;
1219+
static_assert(offsetof(struct create_context, Buffer) == sizeof(struct create_context_hdr),
1220+
"struct member likely outside of __struct_group()");
12191221

12201222
struct smb2_create_req {
12211223
struct smb2_hdr hdr;

0 commit comments

Comments
 (0)