Skip to content

Constant time#591

Closed
thehoul wants to merge 63 commits into
int32-reviewedfrom
int32-repro-cpy
Closed

Constant time#591
thehoul wants to merge 63 commits into
int32-reviewedfrom
int32-repro-cpy

Conversation

@thehoul

@thehoul thehoul commented Dec 12, 2025

Copy link
Copy Markdown
Contributor

copy of #579

ValeriodeSa and others added 30 commits November 24, 2025 11:26
ValeriodeSa and others added 24 commits November 24, 2025 11:26
Change GroupOrder method of edwards25519/constant_scalar.go to return compatible_mod.Mod

Remove saferith usage from edwards25519/constant_scalar.go
add FromString to var_mod.go

add NewUint to var_int.go
use require.NoError() for error checks instead of lenghty if statements
rename comptabile_mod to compatiblemod (lint complaint)
@thehoul
thehoul changed the base branch from master to int32-reviewed December 12, 2025 09:42
// symbol names in the assembly files. See Issue 1.
func TestLinkWithStdlib(t *testing.T) {
bigmod.NewNat()
k, err := rsa.GenerateKey(rand.Reader, 1024)

Check failure

Code scanning / SonarCloud

Cryptographic keys should be robust High test

Use a key length of at least '2048' bits for 'RSA' cipher algorithm. See more on SonarQube Cloud
@thehoul thehoul closed this Dec 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants