Pass whole sats to new_inflight function

This is an alternative to what we discussed on Friday to fix the interop test where we splice-out funds from cln that were received via an htlc payment.
ElementsProject#8021 (comment)

It should be safer to use `amount_msat_to_sat_round_down` instead of `amount_msat_to_sat` because we can drop sub-sat amounts for splices. Also if `amount_msat_to_sat` fails it won't set `local_funds` and create UB.

Author: remyers

lightningd/channel_control.c

@@ -861,23 +861,25 @@ static void handle_add_inflight(struct lightningd *ld,
 		return;
 	}
 
-	inflight = new_inflight(channel,
-				remote_funding,
-				&outpoint,
-				feerate,
-				satoshis,
-				channel->our_funds,
-				psbt,
-				channel->lease_expiry,
-				channel->lease_commit_sig,
-				channel->lease_chan_max_msat,
-				channel->lease_chan_max_ppt,
-				0,
-				AMOUNT_MSAT(0),
-				AMOUNT_SAT(0),
-				splice_amnt,
-				i_am_initiator,
-				force_sign_first);
+	struct amount_sat local_funds = amount_msat_to_sat_round_down(channel->our_msat);
+
+    inflight = new_inflight(channel,
+                remote_funding,
+                &outpoint,
+                feerate,
+                satoshis,
+                local_funds,
+                psbt,
+                channel->lease_expiry,
+                channel->lease_commit_sig,
+                channel->lease_chan_max_msat,
+                channel->lease_chan_max_ppt,
+                0,
+                AMOUNT_MSAT(0),
+                AMOUNT_SAT(0),
+                splice_amnt,
+                i_am_initiator,
+                force_sign_first);
 
 	log_debug(channel->log, "lightningd adding inflight with txid %s",
 		  fmt_bitcoin_txid(tmpctx,