From 9910a732ca95c125d5890d5b3801a54da85b87b2 Mon Sep 17 00:00:00 2001
From: Philipp Hoenisch <philipp@hoenisch.at>
Date: Mon, 6 Sep 2021 09:44:37 +1000
Subject: [PATCH] Subtract minimal fee from punish tx

---
 cfd_protocol/src/lib.rs | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/cfd_protocol/src/lib.rs b/cfd_protocol/src/lib.rs
index 5c21d5a..25cae27 100644
--- a/cfd_protocol/src/lib.rs
+++ b/cfd_protocol/src/lib.rs
@@ -271,7 +271,6 @@ pub fn punish_transaction(
         .expect("to find commit output in commit tx");
     let commit_amount = revoked_commit_tx.output[commit_vout].value;
 
-    // Fixme: need to subtract tx fee otherwise we won't be able to publish this transaction.
     let mut punish_tx = {
         let txid = revoked_commit_tx.txid();
 
@@ -284,7 +283,7 @@ pub fn punish_transaction(
             value: commit_amount,
             script_pubkey: address.script_pubkey(),
         };
-        Transaction {
+        let mut tx = Transaction {
             version: 2,
             lock_time: 0,
             input: vec![TxIn {
@@ -292,7 +291,12 @@ pub fn punish_transaction(
                 ..Default::default()
             }],
             output: vec![output],
-        }
+        };
+
+        let fee = tx.get_size() * MIN_RELAY_FEE as usize;
+        tx.output[0].value = commit_amount - fee as u64;
+
+        tx
     };
 
     let digest = SigHashCache::new(&punish_tx).signature_hash(