golang/go

crypto/ed25519: doc of Sign should warn not to prehash messages

Open

#36.761 aperta il 25 gen 2020

Vedi su GitHub
 (1 commento) (2 reazioni) (0 assegnatari)Go (19.008 fork)batch import
DocumentationNeedsFixhelp wanted

Metriche repository

Star
 (133.883 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Typically, digital signature schemas require the user to first hash the payload and then sign it. Correct me if I am wrong, but this should be true for RSA, DSA and ECDSA.

If my understanding is right, ED25519 seems to be different: it already hashes the message in a way that the collision resistant property is guaranteed even when it becomes feasible to find a collision for SHA512. Thus, prehashing the input messages basically nullifies this collision resistant property of ED25519 itself. In fact, if a possible attacker can efficiently find a collision for the prehashed message, they would be able to carry out a forgery attack.

Also, according to RFC8032 section 8.7 , it would be better to mention to avoid signing large messages.

/cc @FiloSottile @katiehockman

Guida contributor

Direzione di ricerca
La richiesta chiede di aggiungere un avviso nel commento di documentazione della funzione Sign in crypto/ed25519 per sconsigliare la pre hashing dei messaggi, poiché ed25519 esegue internamente l'hashing del messaggio e la pre hashing ne annulla la resistenza alle collisioni. La modifica dovrebbe anche fare riferimento alla sezione 8.7 di RFC8032, che sconsiglia di firmare messaggi di grandi dimensioni. Il file da modificare è probabilmente go/src/crypto/ed25519/ed25519.go.
Tech stack
go
Dominio
backendsecurity
Tipo issue
Documentazione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Gocryptography basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80