3.16.40-rc1 review patch. If anyone has any objections, please let me know. ------------------ From: Steve French <smfrench@xxxxxxxxx> commit 24df1483c272c99ed88b0cba135d0e1dfdee3930 upstream. Cleanup some missing mem frees on some cifs ioctls, and clarify others to make more obvious that no data is returned. Signed-off-by: Steve French <smfrench@xxxxxxxxx> Acked-by: Sachin Prabhu <sprabhu@xxxxxxxxxx> [bwh: Backported to 3.16: - Drop changes to smb2_duplicate_extents(), smb3_set_integrity() - Adjust context] Signed-off-by: Ben Hutchings <ben@xxxxxxxxxxxxxxx> --- --- a/fs/cifs/smb2ops.c +++ b/fs/cifs/smb2ops.c @@ -228,7 +228,7 @@ SMB3_request_interfaces(const unsigned i le64_to_cpu(out_buf->LinkSpeed)); } else cifs_dbg(VFS, "error %d on ioctl to get interface list\n", rc); - + kfree(out_buf); return rc; } #endif /* STATS2 */ @@ -640,6 +640,7 @@ smb2_clone_range(const unsigned int xid, cchunk_out: kfree(pcchunk); + kfree(retbuf); return rc; }