microsoft/msquic

Miss check for the return value of CXPLAT_ALLOC_NONPAGED and CxPlatPoolAlloc

Open

#5.233 geöffnet am 10. Juli 2025

Auf GitHub ansehen
 (0 Kommentare) (1 Reaktion) (2 zugewiesene Personen)C (671 Forks)github user discovery
Area: TestingHowFound: MSRTriagedgood first issuehelp wanted

Repository-Metriken

Stars
 (4.712 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

Description There are several instances in the codebase where the return value of CXPLAT_ALLOC_NONPAGED and CxPlatPoolAlloc is not checked before being used.

https://github.com/microsoft/msquic/blob/a6fe43d7928dc550b2f24599a2769df0358f963d/src/perf/lib/Tcp.cpp#L706-L707

https://github.com/microsoft/msquic/blob/a6fe43d7928dc550b2f24599a2769df0358f963d/src/core/unittest/RecvBufferTest.cpp#L55-L58

https://github.com/microsoft/msquic/blob/a6fe43d7928dc550b2f24599a2769df0358f963d/src/core/unittest/RecvBufferTest.cpp#L77-L79

https://github.com/microsoft/msquic/blob/a6fe43d7928dc550b2f24599a2769df0358f963d/src/core/unittest/RecvBufferTest.cpp#L78-L79

https://github.com/microsoft/msquic/blob/a6fe43d7928dc550b2f24599a2769df0358f963d/src/core/unittest/RecvBufferTest.cpp#L82-L84

https://github.com/microsoft/msquic/blob/a6fe43d7928dc550b2f24599a2769df0358f963d/src/platform/unittest/TlsTest.cpp#L588-L596

Suggested Fix Add a null check immediately after each CXPLAT_ALLOC_NONPAGED and CxPlatPoolAlloc call, and handle the failure case appropriately

Contributor Guide

Research-Richtung
Fügen Sie Nullprüfungen nach jedem CXPLAT ALLOC NONPAGED und CxPlatPoolAlloc Aufruf in den angegebenen Dateien hinzu, um Zuordnungsfehler zu behandeln.
Tech Stack
c
Domain
networkingbackend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
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.
Meist klar
Voraussetzungen
C
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75