diff options
| author | Jeff Layton | 2008-05-13 04:56:05 +0200 |
|---|---|---|
| committer | Steve French | 2008-05-13 05:06:13 +0200 |
| commit | 6353450a2deefaa79cdb4fd2b72830c7db610256 (patch) | |
| tree | 8a09c4b38d90c26ec431cb9e3691a73ca0da2031 /fs/cifs/cifsproto.h | |
| parent | [CIFS] CIFS currently allows for permissions to be changed on files, even (diff) | |
| download | kernel-qcow2-linux-6353450a2deefaa79cdb4fd2b72830c7db610256.tar.gz kernel-qcow2-linux-6353450a2deefaa79cdb4fd2b72830c7db610256.tar.xz kernel-qcow2-linux-6353450a2deefaa79cdb4fd2b72830c7db610256.zip | |
fix memory leak in CIFSFindNext
When CIFSFindNext gets back an -EBADF from a call, it sets the return
code of the function to 0 and eventually exits. Doing this makes the
cleanup at the end of the function skip freeing the SMB buffer, so
we need to make sure we free the buffer explicitly when doing this.
If we don't you end up with errors like this when unplugging the cifs
kernel module:
slab error in kmem_cache_destroy(): cache `cifs_request': Can't free all objects
[<c046bdbf>] kmem_cache_destroy+0x61/0xf3
[<e0f03045>] cifs_destroy_request_bufs+0x14/0x28 [cifs]
[<e0f2016e>] exit_cifs+0x1e/0x80 [cifs]
[<c043aeae>] sys_delete_module+0x192/0x1b8
[<c04451fd>] audit_syscall_entry+0x14b/0x17d
[<c0405413>] syscall_call+0x7/0xb
=======================
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Diffstat (limited to 'fs/cifs/cifsproto.h')
0 files changed, 0 insertions, 0 deletions