From fbccfbbe0ee26a7fe6ed1c6688e83ee63d245d1d Mon Sep 17 00:00:00 2001 From: Mike Hearn Date: Thu, 23 Oct 2014 17:11:29 +0200 Subject: [PATCH] Default risk analysis: fix an off-by-one error in dust output comparisons. --- core/src/main/java/org/bitcoinj/wallet/DefaultRiskAnalysis.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/core/src/main/java/org/bitcoinj/wallet/DefaultRiskAnalysis.java b/core/src/main/java/org/bitcoinj/wallet/DefaultRiskAnalysis.java index da8a0116..455cbb38 100644 --- a/core/src/main/java/org/bitcoinj/wallet/DefaultRiskAnalysis.java +++ b/core/src/main/java/org/bitcoinj/wallet/DefaultRiskAnalysis.java @@ -151,7 +151,7 @@ public class DefaultRiskAnalysis implements RiskAnalysis { * Checks the output to see if the script violates a standardness rule. Not complete. */ public static RuleViolation isOutputStandard(TransactionOutput output) { - if (MIN_ANALYSIS_NONDUST_OUTPUT.compareTo(output.getValue()) > 0) + if (output.getValue().compareTo(MIN_ANALYSIS_NONDUST_OUTPUT) < 0) return RuleViolation.DUST; for (ScriptChunk chunk : output.getScriptPubKey().getChunks()) { if (chunk.isPushData() && !chunk.isShortestPossiblePushData())