Zhu Shangzhong has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464535 ) Change subject: Coverity: check if data->session is NULL after deref ...................................................................... Coverity: check if data->session is NULL after deref In nfs4_op_reclaim_complete, the pointer data->session has been dereferenced in the statement "nfs_client_id_t *clientid = data->session->clientid_record;" at the beginning. The "if" condition "if (data->session == NULL)" is no effect. So the "if" condition may be removed. Change-Id: Ia3d62cc8957aca9e006fa954619126639cdda9fa Signed-off-by: Shangzhong Zhu <zhu.shangzhong(a)zte.com.cn&gt; --- M src/Protocols/NFS/nfs4_op_reclaim_complete.c 1 file changed, 0 insertions(+), 5 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/35/464535/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464535 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Ia3d62cc8957aca9e006fa954619126639cdda9fa Gerrit-Change-Number: 464535 Gerrit-PatchSet: 1 Gerrit-Owner: Zhu Shangzhong <zhu.shangzhong(a)zte.com.cn&gt; Gerrit-MessageType: newchange