Clean this up a bit more.

This commit is contained in:
JoelKatz
2013-06-10 19:29:24 -07:00
parent 3d9a2284ed
commit 78eb8a0cdc

View File

@@ -852,17 +852,16 @@ static void checkTransaction(Job&, int flags, SerializedTransaction::pointer stx
if (isSetBit(flags, SF_SIGGOOD)) if (isSetBit(flags, SF_SIGGOOD))
tx = boost::make_shared<Transaction>(stx, false); tx = boost::make_shared<Transaction>(stx, false);
else else
{
tx = boost::make_shared<Transaction>(stx, true); tx = boost::make_shared<Transaction>(stx, true);
if (tx->getStatus() == INVALID)
{ if (tx->getStatus() == INVALID)
theApp->getSuppression().setFlag(stx->getTransactionID(), SF_BAD); {
Peer::punishPeer(peer, LT_InvalidSignature); theApp->getSuppression().setFlag(stx->getTransactionID(), SF_BAD);
return; Peer::punishPeer(peer, LT_InvalidSignature);
} return;
else
theApp->getSuppression().setFlag(stx->getTransactionID(), SF_SIGGOOD);
} }
else
theApp->getSuppression().setFlag(stx->getTransactionID(), SF_SIGGOOD);
theApp->getOPs().processTransaction(tx, isSetBit(flags, SF_TRUSTED)); theApp->getOPs().processTransaction(tx, isSetBit(flags, SF_TRUSTED));