keepkey, trezor: Handle if ButtonRequest is sent during sendpin

Keepkey is now displaying the passphrase to users after it receives it
and responding with a ButtonRequest message. However this was not being
properly handled by sendpin which would cause the device to hang as it
waited for a ButtonAck. To resolve this, use self.call but avoid the
firmware check which will fail due to the lack of self.features.

Author: achow101

hwilib/devices/trezor.py

@@ -737,7 +737,7 @@ def send_pin(self, pin: str) -> bool:
                     raise DeviceAlreadyUnlockedError('The PIN has already been sent to this device')
             return False
         elif isinstance(resp, messages.PassphraseRequest):
-            pass_resp = self.client.call_raw(messages.PassphraseAck(passphrase=self.client.ui.get_passphrase(available_on_device=False), on_device=False))
+            pass_resp = self.client.call(messages.PassphraseAck(passphrase=self.client.ui.get_passphrase(available_on_device=False), on_device=False), check_fw=False)
             if isinstance(pass_resp, messages.Deprecated_PassphraseStateRequest):
                 self.client.call_raw(messages.Deprecated_PassphraseStateAck())
         return True

hwilib/devices/trezorlib/client.py

@@ -212,8 +212,9 @@ def _callback_button(self, msg):
         return self._raw_read()
 
     @tools.session
-    def call(self, msg):
-        self.check_firmware_version()
+    def call(self, msg, check_fw = True):
+        if check_fw:
+            self.check_firmware_version()
         resp = self.call_raw(msg)
         while True:
             if isinstance(resp, messages.PinMatrixRequest):