From 23423102e70561471cd42db4ac035624d33f0bd0 Mon Sep 17 00:00:00 2001
From: CalDescent <caldescent@protonmail.com>
Date: Sun, 4 Sep 2022 11:51:24 +0100
Subject: [PATCH] Use onlineAccountTimestamp for all mempow hard fork related
 code in OnlineAccountsManager too.

---
 src/main/java/org/qortal/block/Block.java          |  2 +-
 .../qortal/controller/OnlineAccountsManager.java   | 14 +++++++-------
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/main/java/org/qortal/block/Block.java b/src/main/java/org/qortal/block/Block.java
index b60e1acb..0d2deb45 100644
--- a/src/main/java/org/qortal/block/Block.java
+++ b/src/main/java/org/qortal/block/Block.java
@@ -1076,7 +1076,7 @@ public class Block {
 
 			// Validate the rest
 			for (OnlineAccountData onlineAccount : onlineAccounts)
-				if (!OnlineAccountsManager.getInstance().verifyMemoryPoW(onlineAccount, this.blockData.getTimestamp()))
+				if (!OnlineAccountsManager.getInstance().verifyMemoryPoW(onlineAccount))
 					return ValidationResult.ONLINE_ACCOUNT_NONCE_INCORRECT;
 		}
 
diff --git a/src/main/java/org/qortal/controller/OnlineAccountsManager.java b/src/main/java/org/qortal/controller/OnlineAccountsManager.java
index a384dd01..cf309b23 100644
--- a/src/main/java/org/qortal/controller/OnlineAccountsManager.java
+++ b/src/main/java/org/qortal/controller/OnlineAccountsManager.java
@@ -305,8 +305,8 @@ public class OnlineAccountsManager {
         }
 
         // Validate mempow if feature trigger is active
-        if (now >= BlockChain.getInstance().getOnlineAccountsMemoryPoWTimestamp()) {
-            if (!getInstance().verifyMemoryPoW(onlineAccountData, now)) {
+        if (onlineAccountTimestamp >= BlockChain.getInstance().getOnlineAccountsMemoryPoWTimestamp()) {
+            if (!getInstance().verifyMemoryPoW(onlineAccountData)) {
                 LOGGER.trace(() -> String.format("Rejecting online reward-share for account %s due to invalid PoW nonce", mintingAccount.getAddress()));
                 return false;
             }
@@ -542,7 +542,7 @@ public class OnlineAccountsManager {
 
             // Compute nonce
             Integer nonce;
-            if (isMemoryPoWActive(NTP.getTime())) {
+            if (isMemoryPoWActive(onlineAccountsTimestamp)) {
                 try {
                     nonce = this.computeMemoryPoW(mempowBytes, publicKey, onlineAccountsTimestamp);
                     if (nonce == null) {
@@ -565,7 +565,7 @@ public class OnlineAccountsManager {
             OnlineAccountData ourOnlineAccountData = new OnlineAccountData(onlineAccountsTimestamp, signature, publicKey, nonce);
 
             // Make sure to verify before adding
-            if (verifyMemoryPoW(ourOnlineAccountData, NTP.getTime())) {
+            if (verifyMemoryPoW(ourOnlineAccountData)) {
                 ourOnlineAccounts.add(ourOnlineAccountData);
             }
         }
@@ -615,7 +615,7 @@ public class OnlineAccountsManager {
     }
 
     private Integer computeMemoryPoW(byte[] bytes, byte[] publicKey, long onlineAccountsTimestamp) throws TimeoutException {
-        if (!isMemoryPoWActive(NTP.getTime())) {
+        if (!isMemoryPoWActive(onlineAccountsTimestamp)) {
             LOGGER.info("Mempow start timestamp not yet reached, and onlineAccountsMemPoWEnabled not enabled in settings");
             return null;
         }
@@ -641,8 +641,8 @@ public class OnlineAccountsManager {
         return nonce;
     }
 
-    public boolean verifyMemoryPoW(OnlineAccountData onlineAccountData, Long timestamp) {
-        if (!isMemoryPoWActive(timestamp)) {
+    public boolean verifyMemoryPoW(OnlineAccountData onlineAccountData) {
+        if (!isMemoryPoWActive(onlineAccountData.getTimestamp())) {
             // Not active yet, so treat it as valid
             return true;
         }