Skip to content

Commit 9db4d8e

Browse files
sigmoidrAdamj1232
andauthored
Add null check to prevent potential runtime errors (#2319)
* fix: provider['ethereum'] undefined * fix: check provider.providers is array --------- Co-authored-by: Adam Carpenter <adamcarpenter86@gmail.com>
1 parent dd18c61 commit 9db4d8e

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

packages/injected/src/helpers.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ export const isWalletAvailable = (
4040
// For multiple injected providers, check providers array
4141
// example coinbase inj wallet pushes over-ridden wallets
4242
// into a providers array at window.ethereum
43-
return !!provider.providers?.some(provider =>
43+
return Array.isArray(provider.providers) && !!provider.providers?.some(provider =>
4444
checkProviderIdentity({ provider, device })
4545
)
4646
}

packages/injected/src/wallets.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -581,7 +581,7 @@ const bitget: InjectedWalletModule = {
581581
label: ProviderLabel.Bitget,
582582
injectedNamespace: InjectedNameSpace.Bitget,
583583
checkProviderIdentity: ({ provider }) =>
584-
!!provider && !!provider['ethereum'][ProviderIdentityFlag.Bitget],
584+
!!provider && !!provider['ethereum'] && !!provider['ethereum'][ProviderIdentityFlag.Bitget],
585585
getIcon: async () => (await import('./icons/bitget.js')).default,
586586
getInterface: async () => ({
587587
provider: window.bitkeep && window.bitkeep.ethereum

0 commit comments

Comments
 (0)