Skip to content

Add CVE-2026-33845 to .trivyignore#21

Merged
Kaiohz merged 1 commit into
mainfrom
BRIC-18/multiplexed-content-stream
May 6, 2026
Merged

Add CVE-2026-33845 to .trivyignore#21
Kaiohz merged 1 commit into
mainfrom
BRIC-18/multiplexed-content-stream

Conversation

@Kaiohz
Copy link
Copy Markdown
Contributor

@Kaiohz Kaiohz commented May 6, 2026

Ignore DTLS zero-length fragment DoS in libgnutls30. No fixed version available in Debian bookworm. Application does not use DTLS (HTTP/ WebSocket over TCP only).

Ignore DTLS zero-length fragment DoS in libgnutls30. No fixed version
available in Debian bookworm. Application does not use DTLS (HTTP/
WebSocket over TCP only).
Copy link
Copy Markdown
Contributor Author

@Kaiohz Kaiohz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔍 Code Review — .trivyignore

✅ Analyse du CVE

CVE-2026-33845 : Integer underflow dans le reassembly des fragments DTLS de GnuTLS.

  • Type : DoS (Denial of Service) via fragment zero-length avec offset non-zéro
  • Sévérité : Moyenne (CVSS ~5.9)
  • Vecteur : Remote, DTLS handshake malformé

✅ Justification de l'ignorance

Excellente justification dans le commentaire :

  1. Aucun fix disponible dans Debian bookworm (version 3.7.9-2+deb12u6)
  2. Pas d'impact applicatif : FastAPI/HTTP + WebSocket over TCP → pas de DTLS
  3. Review date : 2026-05-06 → traçabilité

📋 Format du fichier

Le format suit les conventions existantes :

  • ✅ Commentaire descriptif
  • ✅ Contexte (version Debian, raison)
  • ✅ Date de review

🔧 Suggestion mineure

Le fichier ne se termine pas par une newline. Convention POSIX recommandée, mais pas bloquant.


📊 Score de qualité : 9/10

Points positifs :

  • Justification claire et complète
  • Date de review pour traçabilité
  • Analyse d'impact applicatif (pas de DTLS)
  • Conformité au format existant

Amélioration suggérée :

  • Ajouter une newline finale (convention POSIX)

Recommandation : ✅ APPROVE — PR prête à merger. L'exception de sécurité est justifiée.

SoluBot review automatique

@Kaiohz Kaiohz merged commit 001734b into main May 6, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant