diff options
author | Cyril Roelandt <tipecaml@gmail.com> | 2014-04-03 18:05:21 -0400 |
---|---|---|
committer | Steve French <smfrench@gmail.com> | 2014-04-16 14:51:46 -0400 |
commit | 8e3ecc87695f4a7e9e217ebd55ca6a39b6a451b8 (patch) | |
tree | de09ad3494f0f58168b8327feee6c97b38a4b6ab /fs/cifs | |
parent | 60977fcc808664f82412bb37da7be17640ba99d9 (diff) |
fs: cifs: remove unused variable.
In SMB2_set_compression(), the "res_key" variable is only initialized to NULL
and later kfreed. It is therefore useless and should be removed.
Found with the following semantic patch:
<smpl>
@@
identifier foo;
identifier f;
type T;
@@
* f(...) {
...
* T *foo = NULL;
... when forall
when != foo
* kfree(foo);
...
}
</smpl>
Signed-off-by: Cyril Roelandt <tipecaml@gmail.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs')
-rw-r--r-- | fs/cifs/smb2pdu.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/cifs/smb2pdu.c b/fs/cifs/smb2pdu.c index 860344701067..3802f8c94acc 100644 --- a/fs/cifs/smb2pdu.c +++ b/fs/cifs/smb2pdu.c | |||
@@ -1352,7 +1352,6 @@ SMB2_set_compression(const unsigned int xid, struct cifs_tcon *tcon, | |||
1352 | u64 persistent_fid, u64 volatile_fid) | 1352 | u64 persistent_fid, u64 volatile_fid) |
1353 | { | 1353 | { |
1354 | int rc; | 1354 | int rc; |
1355 | char *res_key = NULL; | ||
1356 | struct compress_ioctl fsctl_input; | 1355 | struct compress_ioctl fsctl_input; |
1357 | char *ret_data = NULL; | 1356 | char *ret_data = NULL; |
1358 | 1357 | ||
@@ -1365,7 +1364,6 @@ SMB2_set_compression(const unsigned int xid, struct cifs_tcon *tcon, | |||
1365 | 2 /* in data len */, &ret_data /* out data */, NULL); | 1364 | 2 /* in data len */, &ret_data /* out data */, NULL); |
1366 | 1365 | ||
1367 | cifs_dbg(FYI, "set compression rc %d\n", rc); | 1366 | cifs_dbg(FYI, "set compression rc %d\n", rc); |
1368 | kfree(res_key); | ||
1369 | 1367 | ||
1370 | return rc; | 1368 | return rc; |
1371 | } | 1369 | } |