Revert "Update channel_reestablish logic for channel_ready"

This reverts commit 1426949.

Author: jkczyz

lightning/src/ln/channel.rs

@@ -8667,8 +8667,7 @@ where
 	#[rustfmt::skip]
 	pub fn channel_reestablish<L: Deref, NS: Deref>(
 		&mut self, msg: &msgs::ChannelReestablish, logger: &L, node_signer: &NS,
-		chain_hash: ChainHash, their_features: &InitFeatures, user_config: &UserConfig,
-		best_block: &BestBlock,
+		chain_hash: ChainHash, user_config: &UserConfig, best_block: &BestBlock
 	) -> Result<ReestablishResponses, ChannelError>
 	where
 		L::Target: Logger,
@@ -8791,21 +8790,9 @@ where
 		let is_awaiting_remote_revoke = self.context.channel_state.is_awaiting_remote_revoke();
 		let next_counterparty_commitment_number = INITIAL_COMMITMENT_NUMBER - self.context.cur_counterparty_commitment_transaction_number + if is_awaiting_remote_revoke { 1 } else { 0 };
 
-		let our_features = channelmanager::provided_init_features(user_config);
-		let splicing_negotiated =
-			our_features.supports_splicing() && their_features.supports_splicing();
-		let channel_ready = if msg.next_local_commitment_number == 1 && INITIAL_COMMITMENT_NUMBER - self.holder_commitment_point.transaction_number() == 1 && !splicing_negotiated {
+		let channel_ready = if msg.next_local_commitment_number == 1 && INITIAL_COMMITMENT_NUMBER - self.holder_commitment_point.transaction_number() == 1 {
 			// We should never have to worry about MonitorUpdateInProgress resending ChannelReady
 			self.get_channel_ready(logger)
-		} else if splicing_negotiated {
-			// A node:
-			//   - if `option_splice` was negotiated and `your_last_funding_locked` is not
-			//     set in the `channel_reestablish` it received:
-			//     - MUST retransmit `channel_ready`.
-			msg.your_last_funding_locked_txid
-				.is_none()
-				.then(|| ())
-				.and_then(|_| self.get_channel_ready(logger))
 		} else { None };
 
 		if msg.next_local_commitment_number == next_counterparty_commitment_number {

lightning/src/ln/channelmanager.rs

@@ -10983,13 +10983,12 @@ This indicates a bug inside LDK. Please report this error at https://github.com/
 			match peer_state.channel_by_id.entry(msg.channel_id) {
 				hash_map::Entry::Occupied(mut chan_entry) => {
 					if let Some(chan) = chan_entry.get_mut().as_funded_mut() {
-						let features = &peer_state.latest_features;
 						// Currently, we expect all holding cell update_adds to be dropped on peer
 						// disconnect, so Channel's reestablish will never hand us any holding cell
 						// freed HTLCs to fail backwards. If in the future we no longer drop pending
 						// add-HTLCs on disconnect, we may be handed HTLCs to fail backwards here.
 						let responses = try_channel_entry!(self, peer_state, chan.channel_reestablish(
-							msg, &&logger, &self.node_signer, self.chain_hash, features,
+							msg, &&logger, &self.node_signer, self.chain_hash,
 							&self.default_configuration, &*self.best_block.read().unwrap()), chan_entry);
 						let mut channel_update = None;
 						if let Some(msg) = responses.shutdown_msg {