[fix][elm]解决块设备扇区大小校验失败时存在的disk泄漏问题#11507
Conversation
|
👋 感谢您对 RT-Thread 的贡献!Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范,请在你的仓库中执行以下步骤运行代码格式化工作流(如果格式化CI运行失败)。 🛠 操作步骤 | Steps
完成后,提交将自动更新至 如有问题欢迎联系我们,再次感谢您的贡献!💐 |
📌 Code Review Assignment🏷️ Tag: componentsReviewers: Maihuanyi Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2026-06-22 16:22 CST)
📝 Review Instructions
|
|
| logic_nbr[0] = '0' + index; | ||
|
|
||
| /* save device */ | ||
| disk[index] = mnt->dev_id; |
There was a problem hiding this comment.
dfsv1貌似也有这个问题,可以帮忙一起修正下吗?
拉取/合并请求描述:(PR description)
[
为什么提交这份PR (why to submit this PR)
--当扇区大小检查失败时(设备扇区大小大于 FF_MAX_SS ), disk[index] 没有被重置为 RT_NULL ,导致该槽位被永久占用,无法再被使用。
你的解决方案是什么 (what is your solution)
--- 先校验 sector size :在占用 disk 数组槽位之前先检查扇区大小,校验通过后再占用槽位 :只有确认扇区大小没问题后,才执行 disk[index] = mnt->dev_id。
请提供验证的bsp和config (provide the config and bsp)
]
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
代码质量 Code Quality:
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:
#if 0代码,不包含已经被注释了的代码 All redundant code is removed and cleaned up