summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorJulia Lawall <julia@diku.dk>2008-12-22 21:53:40 +0000
committerSteve French <sfrench@us.ibm.com>2008-12-26 02:29:13 +0000
commit359d67d6ad054ae11ad459665fdfb883aca87782 (patch)
treec4cb640ec1692b7f89144fd8bed7e0505a8cad43 /fs
parentc6fbba0546d3ead18d4a623e76e28bcbaa66a325 (diff)
downloadblackbird-op-linux-359d67d6ad054ae11ad459665fdfb883aca87782.tar.gz
blackbird-op-linux-359d67d6ad054ae11ad459665fdfb883aca87782.zip
[CIFS] Remove redundant test
In fs/cifs/cifssmb.c, pLockData is tested for being NULL at the beginning of the function, and not reassigned subsequently. A simplified version of the semantic patch that makes this change is as follows: (http://www.emn.fr/x-info/coccinelle/) Signed-off-by: Julia Lawall <julia@diku.dk> Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs')
-rw-r--r--fs/cifs/cifssmb.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/fs/cifs/cifssmb.c b/fs/cifs/cifssmb.c
index 824df142f288..552642a507c4 100644
--- a/fs/cifs/cifssmb.c
+++ b/fs/cifs/cifssmb.c
@@ -1871,10 +1871,6 @@ CIFSSMBPosixLock(const int xid, struct cifsTconInfo *tcon,
rc = -EIO; /* bad smb */
goto plk_err_exit;
}
- if (pLockData == NULL) {
- rc = -EINVAL;
- goto plk_err_exit;
- }
data_offset = le16_to_cpu(pSMBr->t2.DataOffset);
data_count = le16_to_cpu(pSMBr->t2.DataCount);
if (data_count < sizeof(struct cifs_posix_lock)) {
OpenPOWER on IntegriCloud