From 68b8360cdac532ea57f150242eb1a6129538b7d4 Mon Sep 17 00:00:00 2001
From: JoelKatz <DavidJoelSchwartz@GMail.com>
Date: Tue, 16 Oct 2012 11:47:42 -0700
Subject: [PATCH] Hashes for transaction nodes with metadata were miscomputed.
 This may cause broken nodes not to stay synched with working nodes.

---
 src/SHAMapNodes.cpp | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/SHAMapNodes.cpp b/src/SHAMapNodes.cpp
index 6c02b79a3b..1d7f696231 100644
--- a/src/SHAMapNodes.cpp
+++ b/src/SHAMapNodes.cpp
@@ -301,6 +301,8 @@ SHAMapTreeNode::SHAMapTreeNode(const SHAMapNode& id, const std::vector<unsigned
 		}
 		else if (prefix == sHP_TransactionNode)
 		{
+			if (s.getLength() < 32)
+				throw std::runtime_error("short TXN node");
 			uint256 txID;
 			s.get256(txID, s.getLength() - 32);
 			s.chop(32);
@@ -347,7 +349,11 @@ bool SHAMapTreeNode::updateHash()
 	}
 	else if (mType == tnTRANSACTION_MD)
 	{
-		nh = Serializer::getPrefixHash(sHP_TransactionNode, mItem->peekData());
+		Serializer s(mItem->peekSerializer().getDataLength() + (256 + 32) / 8);
+		s.add32(sHP_TransactionNode);
+		s.addRaw(mItem->peekData());
+		s.add256(mItem->getTag());
+		nh = s.getSHA512Half();
 	}
 	else
 		assert(false);