diff --git a/src/LedgerNode.cpp b/src/LedgerNode.cpp
index 29b659649a..67216fc1db 100644
--- a/src/LedgerNode.cpp
+++ b/src/LedgerNode.cpp
@@ -33,7 +33,7 @@ LedgerStateParms Ledger::writeBack(LedgerStateParms parms, SerializedLedgerEntry
 		return lepCREATED;
 	}
 
-	if(!mAccountStateMap->updateGiveItem(item, false))
+	if (!mAccountStateMap->updateGiveItem(item, false))
 	{
 		assert(false);
 		return lepERROR;
diff --git a/src/SHAMap.cpp b/src/SHAMap.cpp
index 1779d5f326..ad7f42be47 100644
--- a/src/SHAMap.cpp
+++ b/src/SHAMap.cpp
@@ -520,12 +520,12 @@ bool SHAMap::updateGiveItem(SHAMapItem::pointer item, bool isTransaction)
 	boost::recursive_mutex::scoped_lock sl(mLock);
 
 	std::stack<SHAMapTreeNode::pointer> stack = getStack(tag, true);
-	if(stack.empty()) throw SHAMapException(MissingNode);
+	if (stack.empty()) throw SHAMapException(MissingNode);
 
 	SHAMapTreeNode::pointer node=stack.top();
 	stack.pop();
 
-	if (!node->isLeaf() || (node->peekItem()->getTag() == tag) )
+	if (!node->isLeaf() || (node->peekItem()->getTag() != tag) )
 	{
 		assert(false);
 		return false;