]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
os/bluestore/bluestore_tool: compare retval stat() with -1 42219/head
authorKefu Chai <kchai@redhat.com>
Thu, 20 May 2021 05:55:13 +0000 (13:55 +0800)
committerIgor Fedotov <ifed@suse.com>
Fri, 9 Jul 2021 14:45:57 +0000 (17:45 +0300)
commit0428cf3f8a3d8e76212218d21005c84e801e56f8
tree74d533ddb6b148fd4ef39e3a7e59dd173f090557
parentd887bedfa84aa679d4a7bcb5ddb164e3fd89f8fe
os/bluestore/bluestore_tool: compare retval stat() with -1

before this change, stat() is always called to check if the
file specified by --dev-target exists even if this option is not
specified. also, we compare the retval of stat() with ENOENT, while
state() returns -1 on error.

after this change, stat() is called only if --dev-target is specified,
and we compare the retval of stat() with -1 and 0 only, so if
--dev-target option is not specified, the tool still hehaves.

this change addresses a regression introduced by
94a91f54fe30a4dd113fbc1b02bc3f3d52c82a92

Fixes: https://tracker.ceph.com/issues/50891
Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit d4c65a368c9cf35e01604fc3321f867cbe3e4109)
src/os/bluestore/bluestore_tool.cc