]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
os/bluestore/bluestore_tool: compare retval stat() with -1 41676/head
authorKefu Chai <kchai@redhat.com>
Thu, 20 May 2021 05:55:13 +0000 (13:55 +0800)
committerIgor Fedotov <ifedotov@suse.com>
Thu, 3 Jun 2021 12:25:38 +0000 (15:25 +0300)
commit53ccde967116428f62e189f2e547ab89e820a734
treecbe8d222fa60512d11389eafa8b98b16205ef210
parent6cf5998b7ce0cc71657646d861aca5f6fe0c2bc2
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