diff --git a/accounts/scwallet/securechannel.go b/accounts/scwallet/securechannel.go index acafaf967..1650dfbe5 100644 --- a/accounts/scwallet/securechannel.go +++ b/accounts/scwallet/securechannel.go @@ -165,11 +165,7 @@ func (s *SecureChannelSession) Open() error { // The IV is the last bytes returned from the Open APDU. s.iv = response.Data[scSecretLength:] - if err := s.mutuallyAuthenticate(); err != nil { - return err - } - - return nil + return s.mutuallyAuthenticate() } // mutuallyAuthenticate is an internal method to authenticate both ends of the diff --git a/accounts/scwallet/wallet.go b/accounts/scwallet/wallet.go index 0496ccc15..39e80eb3b 100644 --- a/accounts/scwallet/wallet.go +++ b/accounts/scwallet/wallet.go @@ -1060,7 +1060,7 @@ func determinePublicKey(sig, pubkeyX []byte) ([]byte, error) { sig[64] = byte(v) pubkey, err := crypto.Ecrecover(DerivationSignatureHash[:], sig) if err == nil { - if bytes.Compare(pubkey[1:33], pubkeyX) == 0 { + if bytes.Equal(pubkey[1:33], pubkeyX) { return pubkey, nil } } else if v == 1 || err != secp256k1.ErrRecoverFailed { @@ -1077,7 +1077,7 @@ func makeRecoverableSignature(hash, sig, expectedPubkey []byte) ([]byte, error) sig[64] = byte(v) pubkey, err := crypto.Ecrecover(hash, sig) if err == nil { - if bytes.Compare(pubkey, expectedPubkey) == 0 { + if bytes.Equal(pubkey, expectedPubkey) { return sig, nil } } else if v == 1 || err != secp256k1.ErrRecoverFailed {