keystore: remove unused second argument in keystore_mock_unlocked

It's always NULL.

Author: benma

src/keystore.c

@@ -539,20 +539,15 @@ bool keystore_secp256k1_sign(
 }
 
 #ifdef TESTING
-void keystore_mock_unlocked(const uint8_t* seed, size_t seed_len, const uint8_t* bip39_seed)
+void keystore_mock_unlocked(const uint8_t* seed, size_t seed_len)
 {
     _is_unlocked_device = seed != NULL;
     if (seed != NULL) {
         if (_retain_seed(seed, seed_len) != KEYSTORE_OK) {
             Abort("couldn't retain seed");
         }
     }
-    _is_unlocked_bip39 = bip39_seed != NULL;
-    if (bip39_seed != NULL) {
-        if (!_retain_bip39_seed(bip39_seed)) {
-            Abort("couldn't retain bip39 seed");
-        }
-    }
+    _is_unlocked_bip39 = false;
 }
 
 const uint8_t* keystore_test_get_retained_seed_encrypted(size_t* len_out)

src/keystore.h

@@ -188,7 +188,7 @@ USE_RESULT bool keystore_secp256k1_sign(
 /**
  * convenience to mock the keystore state (locked, seed) in tests.
  */
-void keystore_mock_unlocked(const uint8_t* seed, size_t seed_len, const uint8_t* bip39_seed);
+void keystore_mock_unlocked(const uint8_t* seed, size_t seed_len);
 
 const uint8_t* keystore_test_get_retained_seed_encrypted(size_t* len_out);
 const uint8_t* keystore_test_get_retained_bip39_seed_encrypted(size_t* len_out);

src/rust/bitbox02/src/keystore.rs

@@ -219,7 +219,5 @@ pub fn _encrypt_and_store_seed(seed: &[u8], password: &str) -> Result<(), Error>
 
 #[cfg(feature = "testing")]
 pub fn mock_unlocked(seed: &[u8]) {
-    unsafe {
-        bitbox02_sys::keystore_mock_unlocked(seed.as_ptr(), seed.len() as _, core::ptr::null())
-    }
+    unsafe { bitbox02_sys::keystore_mock_unlocked(seed.as_ptr(), seed.len() as _) }
 }