X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=blobdiff_plain;f=lightning%2Fsrc%2Futil%2Fconfig.rs;h=2f65e958f8b60d0f0a0f72d3cd3b7ca1b009085a;hb=29e34c8a10cf813116c9251188a86978cc97e259;hp=a76f71621011d80b55c02fe898e769055427a808;hpb=c180ddd57a28bb75a4ba77732a1417a5482b4575;p=rust-lightning

diff --git a/lightning/src/util/config.rs b/lightning/src/util/config.rs
index a76f7162..2f65e958 100644
--- a/lightning/src/util/config.rs
+++ b/lightning/src/util/config.rs
@@ -249,7 +249,7 @@ impl Default for ChannelHandshakeLimits {
 
 /// Options which apply on a per-channel basis and may change at runtime or based on negotiation
 /// with our counterparty.
-#[derive(Copy, Clone, Debug)]
+#[derive(Copy, Clone, Debug, PartialEq)]
 pub struct ChannelConfig {
 	/// Amount (in millionths of a satoshi) charged per satoshi for payments forwarded outbound
 	/// over the channel.
@@ -345,6 +345,17 @@ impl Default for ChannelConfig {
 	}
 }
 
+impl_writeable_tlv_based!(ChannelConfig, {
+	(0, forwarding_fee_proportional_millionths, required),
+	(2, forwarding_fee_base_msat, required),
+	(4, cltv_expiry_delta, required),
+	(6, max_dust_htlc_exposure_msat, required),
+	// ChannelConfig serialized this field with a required type of 8 prior to the introduction of
+	// LegacyChannelConfig. To make sure that serialization is not compatible with this one, we use
+	// the next required type of 10, which if seen by the old serialization will always fail.
+	(10, force_close_avoidance_max_fee_satoshis, required),
+});
+
 /// Legacy version of [`ChannelConfig`] that stored the static
 /// [`ChannelHandshakeConfig::announced_channel`] and
 /// [`ChannelHandshakeConfig::commit_upfront_shutdown_pubkey`] fields.
@@ -396,9 +407,9 @@ impl ::util::ser::Readable for LegacyChannelConfig {
 		let mut forwarding_fee_base_msat = 0;
 		read_tlv_fields!(reader, {
 			(0, forwarding_fee_proportional_millionths, required),
-			(1, max_dust_htlc_exposure_msat, (default_value, 5_000_000)),
+			(1, max_dust_htlc_exposure_msat, (default_value, 5_000_000u64)),
 			(2, cltv_expiry_delta, required),
-			(3, force_close_avoidance_max_fee_satoshis, (default_value, 1000)),
+			(3, force_close_avoidance_max_fee_satoshis, (default_value, 1000u64)),
 			(4, announced_channel, required),
 			(6, commit_upfront_shutdown_pubkey, required),
 			(8, forwarding_fee_base_msat, required),