Back out RJC's previous changes; they aren't necessary. Failure is dealt with by the destructor, which will _always_ be called from the factory method if Init() fails.

rdf/base/src/nsInMemoryDataSource.cpp

@@ -631,21 +631,11 @@ InMemoryDataSource::Init()
                                    nsnull);
 
     if (! mReverseArcs)
-    {
-    	PL_HashTableDestroy(mForwardArcs);
-    	mForwardArcs = nsnull;
         return NS_ERROR_OUT_OF_MEMORY;
-    }
 
     mLock = PR_NewLock();
     if (! mLock)
-    {
-    	PL_HashTableDestroy(mForwardArcs);
-    	mForwardArcs = nsnull;
-    	PL_HashTableDestroy(mReverseArcs);
-    	mReverseArcs = nsnull;
         return NS_ERROR_OUT_OF_MEMORY;
-    }
 
 #ifdef PR_LOGGING
     if (! gLog)