diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2019-05-01 12:19:20 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2019-05-01 12:19:20 -0700 |
commit | 7e74e235bb31a1fefc28d5303da0718b88627ea8 (patch) | |
tree | f7aeae2186f8366898d140c9c1d29d39b12c35dc /lib/sg_split.c | |
parent | 459e3a21535ae3c7a9a123650e54f5c882b8fcbf (diff) | |
download | linux-7e74e235bb31a1fefc28d5303da0718b88627ea8.tar.gz linux-7e74e235bb31a1fefc28d5303da0718b88627ea8.tar.bz2 linux-7e74e235bb31a1fefc28d5303da0718b88627ea8.zip |
gcc-9: don't warn about uninitialized btrfs extent_type variable
The 'extent_type' variable does seem to be reliably initialized, but
it's _very_ non-obvious, since there's a "goto next" case that jumps
over the normal initialization. That will then always trigger the
"start >= extent_end" test, which will end up never falling through to
the use of that variable.
But the code is certainly not obvious, and the compiler warning looks
reasonable. Make 'extent_type' an int, and initialize it to an invalid
negative value, which seems to be the common pattern in other places.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'lib/sg_split.c')
0 files changed, 0 insertions, 0 deletions