Eliminate memory leak in sodaDatabase.openCollection()

Author: cjbj

src/njsSodaCollection.cpp

@@ -104,7 +104,7 @@ void njsSodaCollection::Init(Local<Object> target)
 NAN_METHOD(njsSodaCollection::New)
 {
     njsSodaCollection *coll = new njsSodaCollection();
-    coll->Wrap (info.Holder());
+    coll->Wrap(info.Holder());
     info.GetReturnValue().Set(info.Holder());
 }
 
@@ -130,7 +130,6 @@ Local<Object> njsSodaCollection::CreateFromBaton(njsBaton *baton)
     coll->dpiSodaDbHandle = baton->dpiSodaDbHandle;
     coll->jsOracledb.Reset(baton->jsOracledb);
     baton->dpiSodaCollHandle = NULL;
-    baton->dpiSodaDbHandle = NULL;
     return scope.Escape(obj);
 }
 

src/njsSodaDatabase.h

@@ -84,7 +84,7 @@ class njsSodaDatabase : public njsCommon {
     static NAN_METHOD(CreateDocument);
 
     bool IsValid() const { return ( true ); } // sodaDB never closes.
-    njsErrorType GetInvalidErrorType () const { return errInvalidConnection ; }
+    njsErrorType GetInvalidErrorType() const { return errSuccess; }
 
     // Constructor
     njsSodaDatabase() : dpiSodaDbHandle(NULL) {}
@@ -95,6 +95,7 @@ class njsSodaDatabase : public njsCommon {
             dpiSodaDb_release(dpiSodaDbHandle);
             dpiSodaDbHandle = NULL;
         }
+        jsOracledb.Reset();
     }
 
     // Member variables