diff options
| author | Eric Biggers <ebiggers@google.com> | 2022-12-23 21:36:28 +0100 |
|---|---|---|
| committer | Eric Biggers <ebiggers@google.com> | 2023-01-10 04:05:47 +0100 |
| commit | 284d5db5f99efa9e3549eb3cba39379d48879db1 (patch) | |
| tree | 992acc4d74a9accc0bdbb1f73b0b4ac2d311346e /fs/verity/fsverity_private.h | |
| parent | fsverity: remove debug messages and CONFIG_FS_VERITY_DEBUG (diff) | |
| download | linux-284d5db5f99efa9e3549eb3cba39379d48879db1.tar.xz linux-284d5db5f99efa9e3549eb3cba39379d48879db1.zip | |
fsverity: use unsigned long for level_start
fs/verity/ isn't consistent with whether Merkle tree block indices are
'unsigned long' or 'u64'. There's no real point to using u64 for them,
though, since (a) a Merkle tree with over ULONG_MAX blocks would only be
needed for a file larger than MAX_LFS_FILESIZE, and (b) for reads, the
status of all Merkle tree blocks has to be tracked in memory.
Therefore, let's make things a bit more efficient on 32-bit systems by
using 'unsigned long[]' for merkle_tree_params::level_start, instead of
'u64[]'. Also, to be extra safe, explicitly check that there aren't
more than ULONG_MAX Merkle tree blocks.
Signed-off-by: Eric Biggers <ebiggers@google.com>
Reviewed-by: Andrey Albershteyn <aalbersh@redhat.com>
Tested-by: Ojaswin Mujoo <ojaswin@linux.ibm.com>
Link: https://lore.kernel.org/r/20221223203638.41293-2-ebiggers@kernel.org
Diffstat (limited to 'fs/verity/fsverity_private.h')
| -rw-r--r-- | fs/verity/fsverity_private.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/verity/fsverity_private.h b/fs/verity/fsverity_private.h index a16038a0ee67..e8b40c8000be 100644 --- a/fs/verity/fsverity_private.h +++ b/fs/verity/fsverity_private.h @@ -52,7 +52,7 @@ struct merkle_tree_params { * Starting block index for each tree level, ordered from leaf level (0) * to root level ('num_levels - 1') */ - u64 level_start[FS_VERITY_MAX_LEVELS]; + unsigned long level_start[FS_VERITY_MAX_LEVELS]; }; /* |