From f1722c2970393dda37ca67bd4a1b66a66b34db73 Mon Sep 17 00:00:00 2001 From: JoelKatz Date: Thu, 9 May 2013 21:59:01 -0700 Subject: [PATCH] Make sure to always set the result, not just the status. --- src/cpp/ripple/NetworkOPs.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/cpp/ripple/NetworkOPs.cpp b/src/cpp/ripple/NetworkOPs.cpp index 795f51832..b7208a606 100644 --- a/src/cpp/ripple/NetworkOPs.cpp +++ b/src/cpp/ripple/NetworkOPs.cpp @@ -324,6 +324,7 @@ Transaction::pointer NetworkOPs::processTransaction(Transaction::pointer trans, if ((newFlags & SF_BAD) != 0) { // cached bad trans->setStatus(INVALID); + trans->setResult(temBAD_SIGNATURE); return trans; } @@ -333,6 +334,7 @@ Transaction::pointer NetworkOPs::processTransaction(Transaction::pointer trans, { cLog(lsINFO) << "Transaction has bad signature"; trans->setStatus(INVALID); + trans->setResult(temBAD_SIGNATURE); theApp->isNewFlag(trans->getID(), SF_BAD); return trans; }