List:Commits« Previous MessageNext Message »
From:Magnus Svensson Date:July 29 2008 9:40am
Subject:bzr commit into mysql-5.1-telco-6.2 branch (msvensson:2636) Bug#38424
View as plain text  
#At file:///data/msvensson/mysql/5.1-telco-6.2/

 2636 Magnus Svensson	2008-07-29
      Bug#38424 Ndb_mgmd resource leak in 'sendStopMgmd' if connection to mgmd fails
modified:
  storage/ndb/src/mgmsrv/MgmtSrvr.cpp

=== modified file 'storage/ndb/src/mgmsrv/MgmtSrvr.cpp'
--- a/storage/ndb/src/mgmsrv/MgmtSrvr.cpp	2008-04-22 19:36:05 +0000
+++ b/storage/ndb/src/mgmsrv/MgmtSrvr.cpp	2008-07-29 09:39:57 +0000
@@ -908,12 +908,14 @@ int MgmtSrvr::sendStopMgmd(NodeId nodeId
     if(ndb_mgm_connect(h,1,0,0))
     {
       DBUG_PRINT("info",("failed ndb_mgm_connect"));
+      ndb_mgm_destroy_handle(&h);
       return SEND_OR_RECEIVE_FAILED;
     }
     if(!restart)
     {
       if(ndb_mgm_stop(h, 1, (const int*)&nodeId) < 0)
       {
+        ndb_mgm_destroy_handle(&h);
         return SEND_OR_RECEIVE_FAILED;
       }
     }
@@ -923,6 +925,7 @@ int MgmtSrvr::sendStopMgmd(NodeId nodeId
       nodes[0]= (int)nodeId;
       if(ndb_mgm_restart2(h, 1, nodes, initialStart, nostart, abort) < 0)
       {
+        ndb_mgm_destroy_handle(&h);
         return SEND_OR_RECEIVE_FAILED;
       }
     }

Thread
bzr commit into mysql-5.1-telco-6.2 branch (msvensson:2636) Bug#38424Magnus Svensson29 Jul