From: Daniel Stenberg <daniel@haxx.se>
Date: Mon, 23 Apr 2012 21:07:40 +0000 (+0200)
Subject: Curl_MD5_init: fix OOM memory leak
X-Git-Tag: curl-7_26_0~27
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=bd9eb30ffdde211574ba62b0ef5e1b75051fffda;p=curl

Curl_MD5_init: fix OOM memory leak

Bug: http://curl.haxx.se/mail/lib-2012-04/0246.html
Reported by: Michael Mueller
---

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;
 }