bitcoinj/bitcoinj

BIP32: Incorrect seed bit-length validation for master key generation

Open

#4 209 ouverte le 17 mai 2026

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Java (2 509 forks)batch import
Enhancementgood first issue

Métriques du dépôt

Stars
 (4 687 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

I found that DeterministicKey createMasterPrivateKey provides input check for byte[] seed .

        checkArgument(seed.length > 8, () ->
                "seed is too short and could be brute forced");

However, the constraints defined in BIP32

Generate a seed byte sequence S of a chosen length (between 128 and 512 bits; 256 bits is advised) from a (P)RNG.

are different. I wonder whether there is a specific reason for this, or whether this effectively 9-byte lower bound was chosen arbitrarily. In any case, I would recommend adhering to the specification as closely as possible.

Found using the bitcoinfuzz tool.

Guide contributeur

Direction de recherche
Examinez la validation actuelle dans createMasterPrivateKey et mettez la à jour pour imposer une longueur de graine comprise entre 16 et 64 octets (128-512 bits) conformément à la spécification BIP32, en veillant à vérifier à la fois les limites minimale et maximale.
Stack technique
java
Domaine
blockchainsecurity
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitJava
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75