From bd9eb30ffdde211574ba62b0ef5e1b75051fffda Mon Sep 17 00:00:00 2001
From: Daniel Stenberg <daniel@haxx.se>
Date: Mon, 23 Apr 2012 23:07:40 +0200
Subject: Curl_MD5_init: fix OOM memory leak

Bug: http://curl.haxx.se/mail/lib-2012-04/0246.html
Reported by: Michael Mueller
---
 lib/md5.c | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

(limited to 'lib')

diff --git a/lib/md5.c b/lib/md5.c
index e58f2c515..8d66d95ec 100644
--- a/lib/md5.c
+++ b/lib/md5.c
@@ -426,9 +426,9 @@ void Curl_md5it(unsigned char *outbuffer, /* 16 bytes */
   MD5_Final(outbuffer, &ctx);
 }
 
-MD5_context * Curl_MD5_init(const MD5_params *md5params)
+MD5_context *Curl_MD5_init(const MD5_params *md5params)
 {
-  MD5_context* ctxt;
+  MD5_context *ctxt;
 
   /* Create MD5 context */
   ctxt = malloc(sizeof *ctxt);
@@ -438,12 +438,14 @@ MD5_context * Curl_MD5_init(const MD5_params *md5params)
 
   ctxt->md5_hashctx = malloc(md5params->md5_ctxtsize);
 
-  if(!ctxt->md5_hashctx)
-    return ctxt->md5_hashctx;
+  if(!ctxt->md5_hashctx) {
+    free(ctxt);
+    return NULL;
+  }
 
   ctxt->md5_hash = md5params;
 
-  (*md5params->md5_init)(ctxt->md5_hashctx);
+  md5params->md5_init(ctxt->md5_hashctx);
 
   return ctxt;
 }
-- 
cgit v1.2.3