openssl/openssl

OPENSSL_cleanse() may crash on some 32bit sparc cpus (sometimes)

Open

#21 898 ouverte le 30 août 2023

Voir sur GitHub
 (8 commentaires) (0 réactions) (0 assignés)C (11 262 forks)batch import
backlog fixbranch: 3.0branch: 3.1branch: 3.2branch: masterhelp wantedtriaged: bug

Métriques du dépôt

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

Description

The (tricky, I like it!) sparc v9 vs. sparc v8 detection code in OPENSSL_cleanse may sometimes go wrong on some CPUs and eroneously go into the v9 branch, where it then crashes on the STX instructions with an invalid op code trap.

The problem is that some CPUs do not implement the rd %asr instructions fully, but instead always read the y register. If, by chance, %y contains 99 at the start of the function, it will detect the CPU as a v9 implementation.

An example of such a CPU is the microSPARC IIep, I originally ran into the crash by doing "openssl speed" on a SparcStation LX, which has a TI TMS390S10.

Guide contributeur

Direction de recherche
Examiner la logique de détection du processeur dans OPENSSL cleanse pour SPARC, en particulier le comportement de l'instruction rd %asr sur les microSPARC IIep et les processeurs similaires. Tester sur du matériel concerné ou QEMU pour reproduire et corriger le plantage.
Stack technique
c
Domaine
cryptography
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.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
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
CSPARC assemblyGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30