setLedgerTransportPreference in the metamask-controller should not be…

… called, or should return immediately, if canUseHardwareWallets() is false Signed-off-by: Akintayo A. Olusegun <[email protected]>
MetaMask · Jan 25, 2023 · 5360f6a · 5360f6a
1 parent 90dee4d
commit 5360f6a
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/app/scripts/metamask-controller.js b/app/scripts/metamask-controller.js
@@ -4415,6 +4415,10 @@ export default class MetamaskController extends EventEmitter {
    * @param {string} transportType - The Ledger transport type.
    */
   async setLedgerTransportPreference(transportType) {
+    if (!this.canUseHardwareWallets()) {
+      return undefined;
+    }
+
     const currentValue =
       this.preferencesController.getLedgerTransportPreference();
     const newValue =