From 2e9ade61b25edb39542183ef9900f2c4b0dfd16d Mon Sep 17 00:00:00 2001
From: Wilson Snyder <wsnyder@wsnyder.org>
Date: Tue, 19 Jan 2010 12:55:34 -0500
Subject: [PATCH] Fix leak in setenvStr, bug184

---
 src/V3Options.cpp | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/src/V3Options.cpp b/src/V3Options.cpp
index 6987d4bc6..e91dd565a 100644
--- a/src/V3Options.cpp
+++ b/src/V3Options.cpp
@@ -320,15 +320,13 @@ void V3Options::setenvStr(const string& envvar, const string& value, const strin
     } else {
 	UINFO(1,"export "<<envvar<<"="<<value<<endl);
     }
-//setenv() replaced by putenv() in MinGW environment. Prototype is different
-//putenv() requires NAME=VALUE format
 #if !defined (__MINGW32__)
     setenv(envvar.c_str(),value.c_str(),true);
 #else
-    int len = value.size() + envvar.size() + 5;
-    char *str = new char[len];
-    sprintf(str,"%s=%s", envvar.c_str(), value.c_str());
-    putenv(str);
+    //setenv() replaced by putenv() in MinGW environment. Prototype is different
+    //putenv() requires NAME=VALUE format
+    string vareq = envvar + "=" + value;
+    putenv(vareq.c_str());
 #endif
 }