diff --git a/LedgerMaster.cpp b/LedgerMaster.cpp
index e22bef6e29..e8fa90f368 100644
--- a/LedgerMaster.cpp
+++ b/LedgerMaster.cpp
@@ -28,7 +28,11 @@ bool LedgerMaster::addHeldTransaction(Transaction::pointer transaction)
 {
 	boost::recursive_mutex::scoped_lock ml(mLock);
 	if(!mHeldTransactionsByID[transaction->getID()])
+	{
 		mHeldTransactionsByID[transaction->getID()]=transaction;
+		return true;
+	}
+	return false;
 }
 
 void LedgerMaster::pushLedger(Ledger::pointer newLedger)
diff --git a/NetworkOPs.cpp b/NetworkOPs.cpp
index 864ad6494c..e27c7076c2 100644
--- a/NetworkOPs.cpp
+++ b/NetworkOPs.cpp
@@ -93,6 +93,7 @@ int NetworkOPs::findTransactionsByDestination(std::list<Transaction::pointer>& t
 	const uint160& destinationAccount, uint32 startLedgerSeq, uint32 endLedgerSeq, int maxTransactions)
 {
 	// WRITEME
+	return 0;
 }
 
 AccountState::pointer NetworkOPs::getAccountState(const uint160& accountID)
diff --git a/Serializer.cpp b/Serializer.cpp
index 79d25e1284..416f73d4e4 100644
--- a/Serializer.cpp
+++ b/Serializer.cpp
@@ -134,6 +134,7 @@ uint160 Serializer::getRIPEMD160(int size) const
 	uint160 ret;
 	if((size<0)||(size>mData.size())) size=mData.size();
 	RIPEMD160(&(mData.front()), size, (unsigned char *) &ret);
+	return ret;
 }
 
 uint256 Serializer::getSHA256(int size) const
@@ -141,6 +142,7 @@ uint256 Serializer::getSHA256(int size) const
 	uint256 ret;
 	if((size<0)||(size>mData.size())) size=mData.size();
 	SHA256(&(mData.front()), size, (unsigned char *) &ret);
+	return ret;
 }
 
 uint256 Serializer::getSHA512Half(int size) const