From: Alexander Strakh Date: Tue, 1 Sep 2009 17:02:24 +0000 (+0000) Subject: [CIFS] Memory leak in ntlmv2 hash calculation X-Git-Tag: upstream/snapshot3+hdmi~17669^2~7 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1b3859bc9e20d764316346665fc93ecea2d2b176;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git [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 Signed-off-by: Steve French --- diff --git a/fs/cifs/cifsencrypt.c b/fs/cifs/cifsencrypt.c index 7c98095..7efe174 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; }