dotnet/runtime

debugshim.cpp opens and gets handles to libraries, but doesn't close or store the handles

Open

#50.442 aperta il 30 mar 2021

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)C# (5445 fork)batch import
area-Diagnostics-coreclrhelp wanted

Metriche repository

Star
 (17.886 star)
Metriche merge PR
 (Merge medio 12g 11h) (661 PR mergiate in 30 g)

Descrizione

debugshim.cpp's CLRDebuggingImpl::OpenVirtualProcess method obtains handles to a bunch of libraries:

https://github.com/dotnet/runtime/blob/b23205f59ab3853e4b6b43f91a7ca3900c0cc7d0/src/coreclr/debug/shim/debugshim.cpp#L172

https://github.com/dotnet/runtime/blob/b23205f59ab3853e4b6b43f91a7ca3900c0cc7d0/src/coreclr/debug/shim/debugshim.cpp#L193

https://github.com/dotnet/runtime/blob/b23205f59ab3853e4b6b43f91a7ca3900c0cc7d0/src/coreclr/debug/shim/debugshim.cpp#L249

But those handles are not kept around and are not used to close the libraries.

Guida contributor

Direzione di ricerca
Identifica gli handle aperti in debugshim.cpp e assicurati che vengano chiusi con CloseHandle o conservati per una successiva pulizia.
Tech stack
ccpp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Windows API
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60