]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
os/bluestore/bluestore_tool: compare retval stat() with -1
authorKefu Chai <kchai@redhat.com>
Thu, 20 May 2021 05:55:13 +0000 (13:55 +0800)
committerIgor Fedotov <ifed@suse.com>
Fri, 16 Jul 2021 12:54:19 +0000 (15:54 +0300)
commitcf2fa23f72f4f6bebca2eaa665527d9b363e9abf
tree8f070fc0f2d3219512663792c67f8cbea7fec754
parentc2f82daea5d2b686ceebbcce8812fc9bc4538b16
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