diff options
| author | Markus Elfring | 2017-08-20 20:22:02 +0200 |
|---|---|---|
| committer | Ilya Dryomov | 2017-09-06 19:56:52 +0200 |
| commit | d37b1d9943d5138b9b2630b7b7082629a82a1386 (patch) | |
| tree | bf904513456933cbd93e22ce0068e2a6e34802a5 /fs/ceph/caps.c | |
| parent | ceph: delete an unnecessary return statement in update_dentry_lease() (diff) | |
| download | kernel-qcow2-linux-d37b1d9943d5138b9b2630b7b7082629a82a1386.tar.gz kernel-qcow2-linux-d37b1d9943d5138b9b2630b7b7082629a82a1386.tar.xz kernel-qcow2-linux-d37b1d9943d5138b9b2630b7b7082629a82a1386.zip | |
ceph: adjust 36 checks for NULL pointers
The script “checkpatch.pl” pointed information out like the following.
Comparison to NULL could be written ...
Thus fix the affected source code places.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Reviewed-by: Yan, Zheng <zyan@redhat.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Diffstat (limited to 'fs/ceph/caps.c')
| -rw-r--r-- | fs/ceph/caps.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/ceph/caps.c b/fs/ceph/caps.c index f1c5691e8e2c..662ada467c32 100644 --- a/fs/ceph/caps.c +++ b/fs/ceph/caps.c @@ -611,7 +611,7 @@ void ceph_add_cap(struct inode *inode, } if (flags & CEPH_CAP_FLAG_AUTH) { - if (ci->i_auth_cap == NULL || + if (!ci->i_auth_cap || ceph_seq_cmp(ci->i_auth_cap->mseq, mseq) < 0) { ci->i_auth_cap = cap; cap->mds_wanted = wanted; @@ -728,7 +728,7 @@ static void __touch_cap(struct ceph_cap *cap) struct ceph_mds_session *s = cap->session; spin_lock(&s->s_cap_lock); - if (s->s_cap_iterator == NULL) { + if (!s->s_cap_iterator) { dout("__touch_cap %p cap %p mds%d\n", &cap->ci->vfs_inode, cap, s->s_mds); list_move_tail(&cap->session_caps, &s->s_caps); |