openssl/openssl

When both psk_ke and psk_dhe_ke exist, there is no verification whether the "key_share" exists.

Open

#25.124 geöffnet am 7. Aug. 2024

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C (11.262 Forks)batch import
help wantedtriaged: bug

Repository-Metriken

Stars
 (30.157 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

version:3.0.9

I have found that when the client extension is configured as follows, the server will choose the "psk_ke" mode for the handshake

  • signature_algorithms
  • supported_versions
  • psk_key_exchange_modes: psk_dhe_ke, psk_ke
  • supported_groups
  • psk

There are two problems here. according to rfc8446-4.2.9:

psk_dhe_ke: PSK with (EC)DHE key establishment. In this mode, the client and server MUST supply "key_share" values as described in Section 4.2.8.

  1. The psk_key_exchange_modes extension contains psk_dhe_ke, but the extension does not contain key_share. It seems that the server should abort the handshake

according to rfc8446-9.2:

If containing a "supported_groups" extension, it MUST also contain a "key_share" extension, and vice versa. An empty KeyShare.client_shares vector is permitted.

  1. because "psk_ke" mode is available. The association between supported_groups and key_share is not verified.

Contributor Guide

Research-Richtung
Untersuchen Sie die Server Handshake Logik in der OpenSSL Zustandsmaschine, insbesondere die Verarbeitung von ClientHello Erweiterungen. Suchen Sie nach der Stelle, an der psk key exchange modes überprüft wird, und stellen Sie sicher, dass bei Vorhandensein von psk dhe ke eine key share Erweiterung zwingend erforderlich ist. Überprüfen Sie außerdem die Anforderung aus RFC 8446 Abschnitt 9.2, dass bei Vorhandensein von supported groups auch key share vorhanden sein muss, selbst wenn der psk ke Modus ausgewählt ist. Mögliche Dateien: ssl/statem/statem srvr.c, ssl/tls13 enc.c. Es müssen Validierungsprüfungen und entsprechende Fehlerwarnungen hinzugefügt werden.
Tech Stack
c
Domain
securityauthentication
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TLS 1.3 protocol knowledgeC programming
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30