diff options
author | Steve French <sfrench@us.ibm.com> | 2008-05-21 22:02:03 -0400 |
---|---|---|
committer | Steve French <sfrench@us.ibm.com> | 2008-05-21 22:02:03 -0400 |
commit | 0d817bc0d6cdd92ff1ab2e98dd5878659a48659c (patch) | |
tree | c20b5b0ff495e7a3a56366e8b3c523692e3acda5 /fs/cifs/cifssmb.c | |
parent | 78b58e549a3098a8c1408d0214bd25e5d5e7a3a3 (diff) |
[CIFS] Remove redundant NULL check
Noticed by Coverity checker.
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs/cifssmb.c')
-rw-r--r-- | fs/cifs/cifssmb.c | 18 |
1 files changed, 6 insertions, 12 deletions
diff --git a/fs/cifs/cifssmb.c b/fs/cifs/cifssmb.c index 7b9938445b07..9b8b4cfdf993 100644 --- a/fs/cifs/cifssmb.c +++ b/fs/cifs/cifssmb.c | |||
@@ -2159,8 +2159,7 @@ copyRetry: | |||
2159 | cFYI(1, ("Send error in copy = %d with %d files copied", | 2159 | cFYI(1, ("Send error in copy = %d with %d files copied", |
2160 | rc, le16_to_cpu(pSMBr->CopyCount))); | 2160 | rc, le16_to_cpu(pSMBr->CopyCount))); |
2161 | } | 2161 | } |
2162 | if (pSMB) | 2162 | cifs_buf_release(pSMB); |
2163 | cifs_buf_release(pSMB); | ||
2164 | 2163 | ||
2165 | if (rc == -EAGAIN) | 2164 | if (rc == -EAGAIN) |
2166 | goto copyRetry; | 2165 | goto copyRetry; |
@@ -2249,8 +2248,7 @@ createSymLinkRetry: | |||
2249 | if (rc) | 2248 | if (rc) |
2250 | cFYI(1, ("Send error in SetPathInfo create symlink = %d", rc)); | 2249 | cFYI(1, ("Send error in SetPathInfo create symlink = %d", rc)); |
2251 | 2250 | ||
2252 | if (pSMB) | 2251 | cifs_buf_release(pSMB); |
2253 | cifs_buf_release(pSMB); | ||
2254 | 2252 | ||
2255 | if (rc == -EAGAIN) | 2253 | if (rc == -EAGAIN) |
2256 | goto createSymLinkRetry; | 2254 | goto createSymLinkRetry; |
@@ -4095,8 +4093,7 @@ getDFSRetry: | |||
4095 | target_nodes, nls_codepage); | 4093 | target_nodes, nls_codepage); |
4096 | 4094 | ||
4097 | GetDFSRefExit: | 4095 | GetDFSRefExit: |
4098 | if (pSMB) | 4096 | cifs_buf_release(pSMB); |
4099 | cifs_buf_release(pSMB); | ||
4100 | 4097 | ||
4101 | if (rc == -EAGAIN) | 4098 | if (rc == -EAGAIN) |
4102 | goto getDFSRetry; | 4099 | goto getDFSRetry; |
@@ -5117,8 +5114,7 @@ setPermsRetry: | |||
5117 | if (rc) | 5114 | if (rc) |
5118 | cFYI(1, ("SetPathInfo (perms) returned %d", rc)); | 5115 | cFYI(1, ("SetPathInfo (perms) returned %d", rc)); |
5119 | 5116 | ||
5120 | if (pSMB) | 5117 | cifs_buf_release(pSMB); |
5121 | cifs_buf_release(pSMB); | ||
5122 | if (rc == -EAGAIN) | 5118 | if (rc == -EAGAIN) |
5123 | goto setPermsRetry; | 5119 | goto setPermsRetry; |
5124 | return rc; | 5120 | return rc; |
@@ -5340,8 +5336,7 @@ QAllEAsRetry: | |||
5340 | } | 5336 | } |
5341 | } | 5337 | } |
5342 | } | 5338 | } |
5343 | if (pSMB) | 5339 | cifs_buf_release(pSMB); |
5344 | cifs_buf_release(pSMB); | ||
5345 | if (rc == -EAGAIN) | 5340 | if (rc == -EAGAIN) |
5346 | goto QAllEAsRetry; | 5341 | goto QAllEAsRetry; |
5347 | 5342 | ||
@@ -5490,8 +5485,7 @@ QEARetry: | |||
5490 | } | 5485 | } |
5491 | } | 5486 | } |
5492 | } | 5487 | } |
5493 | if (pSMB) | 5488 | cifs_buf_release(pSMB); |
5494 | cifs_buf_release(pSMB); | ||
5495 | if (rc == -EAGAIN) | 5489 | if (rc == -EAGAIN) |
5496 | goto QEARetry; | 5490 | goto QEARetry; |
5497 | 5491 | ||