Add warnings reminding users to use SegWit funding inputs

diff --git a/src/ln/channelmanager.rs b/src/ln/channelmanager.rs
index ed69e8a033f1aed61cc14c305a80fe4793f14457..d1e8eefd4a5f5aa3669055d89f9155b92e8f219d 100644 (file)
--- a/src/ln/channelmanager.rs
+++ b/src/ln/channelmanager.rs
@@ -1233,6 +1233,9 @@ impl ChannelManager {
 
        /// Call this upon creation of a funding transaction for the given channel.
        ///
+       /// Note that ALL inputs in the transaction pointed to by funding_txo MUST spend SegWit outputs
+       /// or your counterparty can steal your funds!
+       ///
        /// Panics if a funding transaction has already been provided for this channel.
        ///
        /// May panic if the funding_txo is duplicative with some other channel (note that this should

diff --git a/src/util/events.rs b/src/util/events.rs
index 37cee2eded48778d581f1b808aafbe6ab5989c16..a9a7d06fee51ab86b1c4f1d752eff6f4684ba3fa 100644 (file)
--- a/src/util/events.rs
+++ b/src/util/events.rs
@@ -27,6 +27,8 @@ pub enum Event {
        /// Used to indicate that the client should generate a funding transaction with the given
        /// parameters and then call ChannelManager::funding_transaction_generated.
        /// Generated in ChannelManager message handling.
+       /// Note that *all inputs* in the funding transaction must spend SegWit outputs or your
+       /// counterparty can steal your funds!
        FundingGenerationReady {
                /// The random channel_id we picked which you'll need to pass into
                /// ChannelManager::funding_transaction_generated.