Make sure to always set the result, not just the status.

This commit is contained in:
JoelKatz
2013-05-09 21:59:01 -07:00
parent e9984a932e
commit f1722c2970

View File

@@ -324,6 +324,7 @@ Transaction::pointer NetworkOPs::processTransaction(Transaction::pointer trans,
if ((newFlags & SF_BAD) != 0) if ((newFlags & SF_BAD) != 0)
{ // cached bad { // cached bad
trans->setStatus(INVALID); trans->setStatus(INVALID);
trans->setResult(temBAD_SIGNATURE);
return trans; return trans;
} }
@@ -333,6 +334,7 @@ Transaction::pointer NetworkOPs::processTransaction(Transaction::pointer trans,
{ {
cLog(lsINFO) << "Transaction has bad signature"; cLog(lsINFO) << "Transaction has bad signature";
trans->setStatus(INVALID); trans->setStatus(INVALID);
trans->setResult(temBAD_SIGNATURE);
theApp->isNewFlag(trans->getID(), SF_BAD); theApp->isNewFlag(trans->getID(), SF_BAD);
return trans; return trans;
} }