dotnet/runtime

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

Open

#50,442 创建于 2021年3月30日

在 GitHub 查看
 (4 评论) (0 反应) (0 负责人)C# (5,445 fork)batch import
area-Diagnostics-coreclrhelp wanted

仓库指标

Star
 (17,886 star)
PR 合并指标
 (平均合并 12天 11小时) (30 天内合并 661 个 PR)

描述

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.

贡献者指南

研究方向
识别debugshim.cpp中打开的句柄，并使用CloseHandle正确关闭或妥善存储以便后续清理。
技术栈
ccpp
领域
backend
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
较久未更新
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
C++Windows API
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
60