diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2026-10-13 22:49:17 +0200 |
|---|---|---|
| committer | Namjae Jeon <linkinjeon@kernel.org> | 2026-10-20 17:28:05 +0900 |
| commit | 212e79d58aafdfcfd280806ef38e86dcdc7e2b47 (patch) | |
| tree | 49c645a333fa19b3433e686f26681938bd49826c /include/uapi/linux | |
| parent | 950f860839f31b7617190cf0bdd277abf3e43125 (diff) | |
| download | linux-next-212e79d58aafdfcfd280806ef38e86dcdc7e2b47.tar.gz linux-next-212e79d58aafdfcfd280806ef38e86dcdc7e2b47.zip | |
ntfs: avoid unnecessary padding in uapi structure
Building the uapi headers with -Wpadded shows a warning that
may indicate an incompatibility:
./usr/include/linux/ntfs.h:91:1: error: padding struct size to alignment boundary with 4 bytes [-Werror=padded]
This is relatively harmless here as the 64-bit members force natural
alignment, so this is not a problem for compat ioctls, and I checked
that the padding word is still initialized by means of using kvzalloc()
to allocate it, but it's better to avoid the padding out of
principle.
Change the reserved member to use a 64-bit type so the name field
starts after the structure and all padding is explicit.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Reviewed-by: Baolin Liu <liubaolin@kylinos.cn>
Signed-off-by: Namjae Jeon <linkinjeon@kernel.org>
Diffstat (limited to 'include/uapi/linux')
| -rw-r--r-- | include/uapi/linux/ntfs.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/uapi/linux/ntfs.h b/include/uapi/linux/ntfs.h index f5b3d4997a56..89467ba07262 100644 --- a/include/uapi/linux/ntfs.h +++ b/include/uapi/linux/ntfs.h @@ -86,7 +86,7 @@ struct ntfs_stream_entry { __aligned_u64 alloc_size; __u32 name_len; __u32 name_offset; - __u32 reserved; + __aligned_u64 reserved; __u8 name[]; }; |
