microsoft/msquic

Miss check for the return value of CXPLAT_ALLOC_NONPAGED and CxPlatPoolAlloc

Ouverte

#5 233 ouverte le 10 juil. 2025

Voir sur GitHubGuide contributeur
 (0 commentaire) (1 réaction) (2 personnes assignées)C (687 forks)github user discovery
Area: TestingHowFound: MSRTriagedgood first issuehelp wanted

Métriques du dépôt

Stars
 (4 767 étoiles)
Métriques de merge PR
 (Merge moyen 6j) (40 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Ajoutez des vérifications nulles après chaque appel CXPLAT ALLOC NONPAGED et CxPlatPoolAlloc dans les fichiers spécifiés pour gérer les échecs d'allocation.
Stack technique
c
Domaine
networkingbackend
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.
1-3 heures
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.
Plutôt claire
Prérequis
C
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75