Flag invalid outputs for reserves

src/script/script.cpp

@@ -854,10 +854,17 @@ CCurrencyValueMap CScript::ReserveOutValue(COptCCParams &p, bool spendableOnly)
                 // larger in a way that matters is on testnet for now.
                 CCommitmentHash ch(p.vData[0]);
 
-                // TODO: HARDENING - once Verus Vault activates on mainnet, support currencies
-                if (ch.IsValid() && !_IsVerusMainnetActive())
+                if (ch.IsValid())
                 {
                     retVal = ch.reserveValues;
+
+                    // TODO: HARDENING - once Verus Vault activates on mainnet, support currencies and remove this if statement just below
+                    // until PBaaS, we should have no valid currency outputs on mainnet
+                    if (_IsVerusMainnetActive() && retVal.valueMap.size())
+                    {
+                        LogPrintf("%s: invalid identity commitment output detected\n", __func__);
+                        printf("%s: invalid identity commitment output detected\n", __func__);
+                    }
                 }
             }
         }