Merge pull request #1887 from TheBlueMatt/2022-11-definitely-valid

Remove cryptographically unreachable error conditions

diff --cc lightning/src/ln/chan_utils.rs
index 7fc817995fbdba393809c73c264e6e010a5e27c0,b5b9d2ee77f97c36280cdbbbf6825b838d2ed8db..39106f0c775fee350116d8830fcab915b55d2ee6
--- 1/lightning/src/ln/chan_utils.rs
--- 2/lightning/src/ln/chan_utils.rs
+++ b/lightning/src/ln/chan_utils.rs
@@@ -26,11 -25,10 +26,10 @@@ use crate::ln::msgs::DecodeError
  use crate::util::ser::{Readable, Writeable, Writer};
  use crate::util::{byte_utils, transaction_utils};
  
 -use bitcoin::hash_types::WPubkeyHash;
  use bitcoin::secp256k1::{SecretKey, PublicKey, Scalar};
  use bitcoin::secp256k1::{Secp256k1, ecdsa::Signature, Message};
- use bitcoin::secp256k1::Error as SecpError;
  use bitcoin::{PackedLockTime, secp256k1, Sequence, Witness};
 +use bitcoin::PublicKey as BitcoinPublicKey;
  
  use crate::io;
  use crate::prelude::*;

diff --cc lightning/src/ln/functional_tests.rs
Simple merge