Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Replace libsecp256k1 with secp256k1 #10798

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
fd1fbff
Replace libsecp256k1 with secp256k1
davxy Feb 4, 2022
e200a83
Wipe ecdsa secret key from memory on drop
davxy Feb 4, 2022
0ec1383
Some comments for a known issue
davxy Feb 4, 2022
7165034
Safer core crypto primitives `from_slice` constructor
davxy Feb 7, 2022
9e79db3
Unit test fix
davxy Feb 7, 2022
7a8c4fd
Enable use of global secp256k1 context
davxy Feb 7, 2022
6b66969
Better comments for ecdsa `Pair` drop
davxy Feb 8, 2022
d318328
Replace `libsecp256k1` with `seco256k1` in `beefy-mmr`
davxy Feb 8, 2022
d0f2b0a
Replace `libsecp256k1` with `secp256k1` in FRAME `contracts`benchmarks
davxy Feb 9, 2022
712cf19
Temporary rollback of `beefy-mmr` to libsecp256k1
davxy Feb 10, 2022
be0ea83
Cargo fmt
davxy Feb 10, 2022
f4d525b
Merge branch 'master' into davxy-replace-libsecp256k1-with-secp256k1
davxy Feb 10, 2022
1348472
Rollback of FRAME `contracts` benchmarks to `libsecp256k1`
davxy Feb 10, 2022
548e8b6
Rollback for unrelated changes
davxy Feb 14, 2022
8e23106
Typo fix
davxy Feb 14, 2022
0ec0bd2
Add comments for deprecated `ecdsa_verify` and `secp256k1_ecdsa_recover`
davxy Feb 15, 2022
d3a5ec9
Merge branch 'master' into davxy-replace-libsecp256k1-with-secp256k1
davxy Feb 16, 2022
35549b7
Merge branch 'master' into davxy-replace-libsecp256k1-with-secp256k1
davxy Feb 21, 2022
45ee0d7
Merge branch 'master' into davxy-replace-libsecp256k1-with-secp256k1
davxy Feb 22, 2022
c711f82
Merge branch 'master' into davxy-replace-libsecp256k1-with-secp256k1
davxy Feb 23, 2022
5442124
Merge branch 'master' into davxy-replace-libsecp256k1-with-secp256k1
davxy Feb 26, 2022
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 21 additions & 1 deletion Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -265,6 +265,7 @@ percent-encoding = { opt-level = 3 }
primitive-types = { opt-level = 3 }
ring = { opt-level = 3 }
rustls = { opt-level = 3 }
secp256k1 = { opt-level = 3 }
sha2 = { opt-level = 3 }
sha3 = { opt-level = 3 }
smallvec = { opt-level = 3 }
Expand Down
2 changes: 1 addition & 1 deletion client/authority-discovery/src/tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ fn cryptos_are_compatible() {
let sp_core_signature = sp_core_secret.sign(message); // no error expected...

assert!(sp_core::ed25519::Pair::verify(
&sp_core::ed25519::Signature::from_slice(&libp2p_signature),
&sp_core::ed25519::Signature::from_slice(&libp2p_signature).unwrap(),
message,
&sp_core_public
));
Expand Down
1 change: 0 additions & 1 deletion client/executor/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ sc-executor-common = { version = "0.10.0-dev", path = "common" }
sc-executor-wasmi = { version = "0.10.0-dev", path = "wasmi" }
sc-executor-wasmtime = { version = "0.10.0-dev", path = "wasmtime", optional = true }
parking_lot = "0.12.0"
libsecp256k1 = "0.7"
sp-core-hashing-proc-macro = { version = "4.0.0-dev", path = "../../primitives/core/hashing/proc-macro" }
lru = "0.6.6"
tracing = "0.1.29"
Expand Down
5 changes: 4 additions & 1 deletion primitives/core/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -54,8 +54,9 @@ schnorrkel = { version = "0.9.1", features = [
"u64_backend",
], default-features = false, optional = true }
hex = { version = "0.4", default-features = false, optional = true }
libsecp256k1 = { version = "0.7", default-features = false, features = ["hmac", "static-context"], optional = true }
libsecp256k1 = { version = "0.7", default-features = false, features = ["static-context"], optional = true }
merlin = { version = "2.0", default-features = false, optional = true }
secp256k1 = { version = "0.21.2", default-features = false, features = ["recovery", "global-context"], optional = true }
ss58-registry = { version = "1.11.0", default-features = false }
sp-core-hashing = { version = "4.0.0", path = "./hashing", default-features = false, optional = true }
sp-runtime-interface = { version = "5.0.0", default-features = false, path = "../runtime-interface" }
Expand Down Expand Up @@ -106,6 +107,7 @@ std = [
"schnorrkel/std",
"regex",
"num-traits/std",
"secp256k1/std",
"sp-core-hashing/std",
"sp-debug-derive/std",
"sp-externalities",
Expand All @@ -129,6 +131,7 @@ full_crypto = [
"schnorrkel",
"hex",
"libsecp256k1",
"secp256k1",
"sp-core-hashing",
"sp-runtime-interface/disable_target_static_assertions",
"merlin",
Expand Down
Loading