Commit 0d817bc0 authored by Steve French's avatar Steve French

[CIFS] Remove redundant NULL check

Noticed by Coverity checker.
Signed-off-by: default avatarSteve French <sfrench@us.ibm.com>
parent 78b58e54
......@@ -2159,8 +2159,7 @@ CIFSSMBCopy(const int xid, struct cifsTconInfo *tcon, const char *fromName,
cFYI(1, ("Send error in copy = %d with %d files copied",
rc, le16_to_cpu(pSMBr->CopyCount)));
}
if (pSMB)
cifs_buf_release(pSMB);
cifs_buf_release(pSMB);
if (rc == -EAGAIN)
goto copyRetry;
......@@ -2249,8 +2248,7 @@ CIFSUnixCreateSymLink(const int xid, struct cifsTconInfo *tcon,
if (rc)
cFYI(1, ("Send error in SetPathInfo create symlink = %d", rc));
if (pSMB)
cifs_buf_release(pSMB);
cifs_buf_release(pSMB);
if (rc == -EAGAIN)
goto createSymLinkRetry;
......@@ -4095,8 +4093,7 @@ CIFSGetDFSRefer(const int xid, struct cifsSesInfo *ses,
target_nodes, nls_codepage);
GetDFSRefExit:
if (pSMB)
cifs_buf_release(pSMB);
cifs_buf_release(pSMB);
if (rc == -EAGAIN)
goto getDFSRetry;
......@@ -5117,8 +5114,7 @@ CIFSSMBUnixSetPerms(const int xid, struct cifsTconInfo *tcon,
if (rc)
cFYI(1, ("SetPathInfo (perms) returned %d", rc));
if (pSMB)
cifs_buf_release(pSMB);
cifs_buf_release(pSMB);
if (rc == -EAGAIN)
goto setPermsRetry;
return rc;
......@@ -5340,8 +5336,7 @@ CIFSSMBQAllEAs(const int xid, struct cifsTconInfo *tcon,
}
}
}
if (pSMB)
cifs_buf_release(pSMB);
cifs_buf_release(pSMB);
if (rc == -EAGAIN)
goto QAllEAsRetry;
......@@ -5490,8 +5485,7 @@ ssize_t CIFSSMBQueryEA(const int xid, struct cifsTconInfo *tcon,
}
}
}
if (pSMB)
cifs_buf_release(pSMB);
cifs_buf_release(pSMB);
if (rc == -EAGAIN)
goto QEARetry;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment