diff options
| author |   Alexander Strakh <strakh@ispras.ru> | 2009-09-01 17:02:24 +0000 |
|---|---|---|
| committer |   Steve French <sfrench@us.ibm.com> | 2009-09-01 17:02:24 +0000 |
| commit | 1b3859bc9e20d764316346665fc93ecea2d2b176 (patch) | |
| tree | 9a643e0bf74d9b8339ae4bc45df1988ed306dc4d /fs/cifs | |
| parent | [CIFS] potential NULL dereference in parse_DFS_referrals() (diff) | |
| download | linux-dev-1b3859bc9e20d764316346665fc93ecea2d2b176.tar.xz linux-dev-1b3859bc9e20d764316346665fc93ecea2d2b176.zip | |
[CIFS] Memory leak in ntlmv2 hash calculation
in function calc_ntlmv2_hash memory is not released.
1. If in the line 333 we successfully allocate memory and assign it to
pctxt variable:
pctxt = kmalloc(sizeof(struct HMACMD5Context), GFP_KERNEL);
then we go to line 376 and exit wihout releasing memory pointed to by pctxt
variable.
Add a memory releasing for pctxt variable before exit from function
calc_ntlmv2_hash.
Signed-off-by: Alexander Strakh <strakh@ispras.ru>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs')
| -rw-r--r-- | fs/cifs/cifsencrypt.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/cifs/cifsencrypt.c b/fs/cifs/cifsencrypt.c index 7c9809523f42..7efe1745494d 100644 --- a/fs/cifs/cifsencrypt.c +++ b/fs/cifs/cifsencrypt.c @@ -373,6 +373,7 @@ calc_exit_2: compare with the NTLM example */ hmac_md5_final(ses->server->ntlmv2_hash, pctxt); + kfree(pctxt); return rc; } |