Switch all callers from poly1305_auth to Poly1305 class
Needs ReviewPublic
Actions

Authored by PiRK on Tue, Dec 2, 13:34.

Details

Reviewers

None

Group Reviewers

Restricted Project

Summary

This also removes the old poly1305_auth interface, as it no longer serves any
function. The new Poly1305 class based interface is more modern and safe.

This concludes backport of core#27993
https://github.com/bitcoin/bitcoin/pull/27993/commits/4e5c933f6a40c07d1c68115f7979b89a5b2ba580

Depends on D19015

Test Plan

ninja all check-all bench-bitcoin bitcoin-fuzzers

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 35185
Build 69828: Build Diff	lint-circular-dependencies · build-without-wallet · build-fuzzer · build-diff · build-debug · build-clang-tidy · build-clang · build-bench
Build 69827: arc lint + arc unit

Event Timeline

PiRK created this revision.Tue, Dec 2, 13:34

Herald added a reviewer: Restricted Project. · View Herald TranscriptTue, Dec 2, 13:34

PiRK requested review of this revision.Tue, Dec 2, 13:34

teamcity edited the summary of this revision. (Show Details)Tue, Dec 2, 13:34

Harbormaster completed remote builds in B35185: Diff 56798.Tue, Dec 2, 13:44

Build Bitcoin ABC Diffs / Diff Testing (build-clang-tidy) failed.

Tail of the build log:

   Compiling httpdate v1.0.3
   Compiling sync_wrapper v1.0.2
   Compiling foldhash v0.1.5
   Compiling postcard v1.1.2
   Compiling chronik-plugin v0.1.0 (/work/chronik/chronik-plugin)
   Compiling url v2.5.4
   Compiling tungstenite v0.24.0
   Compiling event-listener-strategy v0.5.4
   Compiling chronik-util v0.1.0 (/work/chronik/chronik-util)
   Compiling seahash v4.1.0
   Compiling mime v0.3.17
   Compiling topo_sort v0.4.0
   Compiling karyon_jsonrpc_macro v0.6.1
   Compiling axum-core v0.4.5
   Compiling async-channel v2.3.1
   Compiling hyper v1.6.0
   Compiling async-tungstenite v0.29.1
   Compiling karyon_core v0.6.1
   Compiling tokio-tungstenite v0.24.0
   Compiling tower v0.5.2
   Compiling serde_path_to_error v0.1.17
   Compiling serde_urlencoded v0.7.1
   Compiling futures-executor v0.3.31
   Compiling hyper-util v0.1.14
   Compiling base64 v0.22.1
   Compiling matchit v0.7.3
   Compiling futures v0.3.31
   Compiling chronik_lib v0.1.0 (/work/chronik/chronik-lib)
   Compiling tower-http v0.5.2
   Compiling webpki-roots v1.0.1
   Compiling axum v0.7.9
   Compiling rustls-webpki v0.103.3
   Compiling tokio-rustls v0.26.2
   Compiling karyon_async_rustls v0.6.1
   Compiling karyon_net v0.6.1
   Compiling karyon_jsonrpc v0.6.1
   Compiling rocksdb v0.21.0
   Compiling chronik-db v0.1.0 (/work/chronik/chronik-db)
   Compiling chronik-indexer v0.1.0 (/work/chronik/chronik-indexer)
   Compiling chronik-http v0.1.0 (/work/chronik/chronik-http)
    Finished `release` profile [optimized] target(s) in 2m 23s
[168/599] Building native src/secp256k1/gen_context
[1/2] Building C object src/secp256k1/CMakeFiles/gen_context.dir/src/gen_context.c.o
[2/2] Linking C executable src/secp256k1/gen_context
[169/599] Running utility command for _cargo-build_chronik_lib
[171/599] Generating src/ecmult_static_context.h
[172/599] Building C object src/secp256k1/CMakeFiles/recover-bench.dir/src/bench_recover.c.o
[173/599] Building C object src/secp256k1/CMakeFiles/sign-bench.dir/src/bench_sign.c.o
[174/599] Building C object src/secp256k1/CMakeFiles/verify-bench.dir/src/bench_verify.c.o
[175/599] Building C object src/secp256k1/CMakeFiles/secp256k1.dir/src/secp256k1.c.o
[176/599] Linking C static library src/secp256k1/libsecp256k1.a
[177/599] Linking C executable src/secp256k1/recover-bench
[178/599] Linking C executable src/secp256k1/sign-bench
[179/599] Linking C executable src/secp256k1/verify-bench
[180/599] Building C object src/secp256k1/CMakeFiles/internal-bench.dir/src/bench_internal.c.o
[181/599] Building C object src/secp256k1/CMakeFiles/ecmult-bench.dir/src/bench_ecmult.c.o
[182/599] Linking C executable src/secp256k1/internal-bench
[183/599] Linking C executable src/secp256k1/ecmult-bench
ninja: build stopped: cannot make progress due to previous errors.
Build build-clang-tidy failed with exit code 1

PiRK added a child revision: D19017: crypto: remove outdated variant of ChaCha20Poly1305 AEAD.Tue, Dec 2, 14:44