[ADMPORTAL] Connection leak fix 39/2139/1
authorTimoney, Daniel (dt5972) <dtimoney@att.com>
Tue, 14 Mar 2017 15:10:54 +0000 (15:10 +0000)
committerDan Timoney <dtimoney@att.com>
Tue, 14 Mar 2017 20:30:47 +0000 (20:30 +0000)
Added connection.release()

Change-Id: I1caeb4e9ea69099e37e8dc0c3869e1100699b40b
Signed-off-by: Timoney, Daniel (dt5972) <dtimoney@att.com>
(cherry picked from commit 02fb35fa2db2cc0ed8e8f3a735b41d8aacc0f383 [formerly 18e4ba71f5b4c553e045c6d02e1a0a5a000a2d6b])

Former-commit-id: 93a07e83588512d9d75ccffbc532f232ef34f121

admportal/server/router/routes/dbRoutes.js

index d2f40b5..34a90c7 100644 (file)
@@ -275,11 +275,13 @@ exports.saveUser = function(req,res){
 
                connection.query(sql, function(err,result){
                        if(err){
+                               connection.release();
                                res.render("pages/signup", {result:{code:'error', msg:"Unable to get database connection. " + String(err)},header:process.env.MAIN_MENU});
                                return;
                        }
                        if (result.length == 1 || result.length > 1)
                        {
+                               connection.release();
                                res.render("pages/signup", {result:{code:'error', msg:'User Information already exists.'},header:process.env.MAIN_MENU});
                                return;
                        }