diff options
| author | Kent Overstreet <kent.overstreet@gmail.com> | 2021-08-17 21:03:53 +0200 |
|---|---|---|
| committer | Kent Overstreet <kent.overstreet@linux.dev> | 2023-10-22 23:09:10 +0200 |
| commit | 8ddef4d6ccedcd571c9b81f6cd8dff8ddcdb918a (patch) | |
| tree | a051a863be2a1eb82e32b9a372933db8012f0730 | |
| parent | bcachefs: Minor btree iter refactoring (diff) | |
| download | linux-8ddef4d6ccedcd571c9b81f6cd8dff8ddcdb918a.tar.xz linux-8ddef4d6ccedcd571c9b81f6cd8dff8ddcdb918a.zip | |
bcachefs: Fix a valgrind conditional jump
Valgrind was complaining about a jump depending on uninitialized memory
- we weren't, but this change makes the code less confusing for valgrind
to follow.
Signed-off-by: Kent Overstreet <kent.overstreet@gmail.com>
| -rw-r--r-- | fs/bcachefs/varint.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/bcachefs/varint.c b/fs/bcachefs/varint.c index 6955ff5dc19c..e87da470c581 100644 --- a/fs/bcachefs/varint.c +++ b/fs/bcachefs/varint.c @@ -97,7 +97,7 @@ int bch2_varint_encode_fast(u8 *out, u64 v) int bch2_varint_decode_fast(const u8 *in, const u8 *end, u64 *out) { u64 v = get_unaligned_le64(in); - unsigned bytes = ffz(v & 255) + 1; + unsigned bytes = ffz(*in) + 1; if (unlikely(in + bytes > end)) return -1; |