diff options
| author | Julia Lawall <julia@diku.dk> | 2008-12-22 22:53:40 +0100 |
|---|---|---|
| committer | Steve French <sfrench@us.ibm.com> | 2008-12-26 03:29:13 +0100 |
| commit | 359d67d6ad054ae11ad459665fdfb883aca87782 (patch) | |
| tree | c4cb640ec1692b7f89144fd8bed7e0505a8cad43 /fs/cifs | |
| parent | [CIFS] make sure that DFS pathnames are properly formed (diff) | |
| download | linux-359d67d6ad054ae11ad459665fdfb883aca87782.tar.xz 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/cifs')
| -rw-r--r-- | fs/cifs/cifssmb.c | 4 |
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)) { |