]> git.apps.os.sepia.ceph.com Git - xfsprogs-dev.git/commit
xfs_db: fix broken logic in error path
authorDarrick J. Wong <djwong@kernel.org>
Thu, 27 Apr 2023 19:02:33 +0000 (12:02 -0700)
committerCarlos Maiolino <cem@kernel.org>
Wed, 10 May 2023 13:13:27 +0000 (15:13 +0200)
commitf4e6c2435a673b581a4d1b6fdf12d5fe21a6cd61
tree15e1b00ed9703d41daedd68b37533ed332122756
parent002b5d96f1d87620db5cc3b7fa7c690a20d5e7f5
xfs_db: fix broken logic in error path

smatch complains proceeding into the if body if leaf is a null pointer:

check.c:3614 process_leaf_node_dir_v2_int() warn: variable dereferenced before check 'leaf' (see line 3518)

However, the logic here is misleading and broken -- what we're trying to
do is switch between the v4 and v5 variants of the directory check.
We're using @leaf3 being a null pointer (or not) to determine v4 vs. v5,
so the "!" part of the comparison is correct, but the variable used
(leaf) is not.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>
Signed-off-by: Carlos Maiolino <cem@kernel.org>
db/check.c