Re: [PATCH 0/3] Ceph: Adjustments for some function implementations

From: Yan, Zheng
Date: Sun Aug 20 2017 - 21:46:46 EST



> On 21 Aug 2017, at 02:37, SF Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> wrote:
>
> From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
> Date: Sun, 20 Aug 2017 20:32:10 +0200
>
> Three update suggestions were taken into account
> from static source code analysis.
>
> Markus Elfring (3):
> Delete an error message for a failed memory allocation in __get_or_create_frag()
> Delete an unnecessary return statement in update_dentry_lease()
> Adjust 36 checks for null pointers
>
> fs/ceph/addr.c | 2 +-
> fs/ceph/cache.c | 2 +-
> fs/ceph/caps.c | 4 ++--
> fs/ceph/debugfs.c | 2 +-
> fs/ceph/file.c | 2 +-
> fs/ceph/inode.c | 14 +++++---------
> fs/ceph/mds_client.c | 22 +++++++++++-----------
> fs/ceph/mdsmap.c | 6 +++---
> fs/ceph/super.c | 18 +++++++++---------
> fs/ceph/xattr.c | 8 ++++----
> 10 files changed, 38 insertions(+), 42 deletions(-)
>
> â

Whole series applied, thanks

Yan, Zheng

> 2.14.0
>