bitcoin: enable message signing on testnet and regtest

Fixes #1376

Author: benma

CHANGELOG.md

@@ -7,6 +7,7 @@ customers cannot upgrade their bootloader, its changes are recorded separately.
 ## Firmware
 
 ### [Unreleased]
+- Bitcoin: enable message signing on testnet and regtest
 
 ### 9.22.0
 - Update manufacturer HID descriptor to bitbox.swiss

CMakeLists.txt

@@ -91,8 +91,8 @@ endif()
 #
 # Versions MUST contain three parts and start with lowercase 'v'.
 # Example 'v1.0.0'. They MUST not contain a pre-release label such as '-beta'.
-set(FIRMWARE_VERSION "v9.22.0")
-set(FIRMWARE_BTC_ONLY_VERSION "v9.22.0")
+set(FIRMWARE_VERSION "v9.23.0")
+set(FIRMWARE_BTC_ONLY_VERSION "v9.23.0")
 set(BOOTLOADER_VERSION "v1.1.0")
 
 find_package(PythonInterp 3.6 REQUIRED)

py/bitbox02/bitbox02/bitbox02/bitbox02.py

@@ -607,6 +607,8 @@ def btc_sign_msg(
         # pylint: disable=no-member
 
         self._require_atleast(semver.VersionInfo(9, 2, 0))
+        if coin in (btc.TBTC, btc.RBTC):
+            self._require_atleast(semver.VersionInfo(9, 23, 0))
 
         request = btc.BTCRequest()
         request.sign_message.CopyFrom(

src/rust/bitbox02-rust/src/hww/api/bitcoin/signmsg.rs

@@ -36,7 +36,7 @@ const MAX_MESSAGE_SIZE: usize = 1024;
 /// compact format (R and S values), and the last byte is the recoverable id (recid).
 pub async fn process(request: &pb::BtcSignMessageRequest) -> Result<Response, Error> {
     let coin = BtcCoin::try_from(request.coin)?;
-    if coin != BtcCoin::Btc {
+    if !matches!(coin, BtcCoin::Btc | BtcCoin::Tbtc | BtcCoin::Rbtc) {
         return Err(Error::InvalidInput);
     }
     let (keypath, simple_type) = match &request.script_config {
@@ -182,6 +182,52 @@ mod tests {
         );
     }
 
+    #[test]
+    pub fn test_p2wpkh_testnet() {
+        let request = pb::BtcSignMessageRequest {
+            coin: BtcCoin::Tbtc as _,
+            script_config: Some(pb::BtcScriptConfigWithKeypath {
+                script_config: Some(pb::BtcScriptConfig {
+                    config: Some(Config::SimpleType(SimpleType::P2wpkh as _)),
+                }),
+                keypath: vec![84 + HARDENED, 1 + HARDENED, 0 + HARDENED, 0, 0],
+            }),
+            msg: MESSAGE.to_vec(),
+            host_nonce_commitment: None,
+        };
+
+        static mut CONFIRM_COUNTER: u32 = 0;
+
+        mock(Data {
+            ui_confirm_create: Some(Box::new(|params| {
+                match unsafe {
+                    CONFIRM_COUNTER += 1;
+                    CONFIRM_COUNTER
+                } {
+                    1 => {
+                        assert_eq!(params.title, "Sign message");
+                        assert_eq!(params.body, "Coin: BTC Testnet");
+                        true
+                    }
+                    2 => {
+                        assert_eq!(params.title, "Address");
+                        assert_eq!(params.body, "tb1qnlyrq9pshg0v0lsuudjgga4nvmjxhcvketqwdg");
+                        true
+                    }
+                    3 => {
+                        assert_eq!(params.title, "Sign message");
+                        assert_eq!(params.body.as_bytes(), MESSAGE);
+                        true
+                    }
+                    _ => panic!("too many user confirmations"),
+                }
+            })),
+            ..Default::default()
+        });
+        mock_unlocked();
+        assert!(block_on(process(&request)).is_ok());
+    }
+
     #[test]
     pub fn test_p2wpkh_p2sh() {
         let request = pb::BtcSignMessageRequest {
@@ -419,5 +465,24 @@ mod tests {
             })),
             Err(Error::InvalidInput)
         );
+        // Invalid keypath (mainnet keypath on testnet)
+        mock(Data {
+            ..Default::default()
+        });
+        mock_unlocked();
+        assert_eq!(
+            block_on(process(&pb::BtcSignMessageRequest {
+                coin: BtcCoin::Tbtc as _,
+                script_config: Some(pb::BtcScriptConfigWithKeypath {
+                    script_config: Some(pb::BtcScriptConfig {
+                        config: Some(Config::SimpleType(SimpleType::P2wpkh as _))
+                    }),
+                    keypath: vec![84 + HARDENED, 0 + HARDENED, 0 + HARDENED, 0, 0],
+                }),
+                msg: MESSAGE.to_vec(),
+                host_nonce_commitment: None,
+            })),
+            Err(Error::InvalidInput)
+        );
     }
 }