This fixes a couple of problems that Dan spotted with a static checker. Additionally the first patch also simplifies some of the error handling in other callers. Originally, I was going to squash these into the earlier patches that introduced those bugs, but this is new code that isn't enabled by default and Ilya was planning to a PR soon, so it's probably best to just handle them as follow-on fixes to what's currently in ceph-client/master. Jeff Layton (2): ceph: have ceph_mdsc_free_path ignore ERR_PTR values ceph: initialize base and pathlen variables in async dirops cb's fs/ceph/debugfs.c | 4 ---- fs/ceph/dir.c | 4 ++-- fs/ceph/file.c | 4 ++-- fs/ceph/mds_client.c | 6 ++---- fs/ceph/mds_client.h | 2 +- 5 files changed, 7 insertions(+), 13 deletions(-) -- 2.25.2