AlphaWallet / AlphaWallet/alpha-wallet-android

Should show an error, instead of silently importing a invalid private key and then fail when signing

Đang mở
#2,689 5 bình luận 0 reaction 1 người được giao Được @JamesSmartCell nhận Xem trên GitHub
Bug High Priority
Ngôn ngữ chính
Java
Star
646
Fork
579
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

Private keys for the secp256k1 curve must be smaller than the order n, which is `fffffffffffffffffffffffffffffffebaaedce6af48a03bbfd25e8cd0364141` and must not be `0…0` so these keys are invalid:

```
fffffffffffffffffffffffffffffffebaaedce6af48a03bbfd25e8cd0364141
fffffffffffffffffffffffffffffffebaaedce6af48a03bbfd25e8cd0364142
0000000000000000000000000000000000000000000000000000000000000000
```

While this is valid:

```
fffffffffffffffffffffffffffffffebaaedce6af48a03bbfd25e8cd0364140
```

Should show an error, instead of silently importing the invalid private key and then fail when signing

Context: https://canary.discord.com/channels/548357107267928064/991499612617719848/991562068975161426

(on iOS, it crashes, which isn't good either https://github.com/AlphaWallet/alpha-wallet-ios/issues/4921)

Test cases used for iOS: https://github.com/AlphaWallet/alpha-wallet-ios/pull/4926/files#diff-ba15c35c2c0ace5c3cb860323254268977e248980976a3480334bd4078d751d2

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.