代码拉取完成,页面将自动刷新
48
修复潜在的内存泄露风险
已关闭
拉取/合并请求描述:
[
当第25行, malloc失败的时, 该list_dir直接return了, 并没有释放之前opendir申请的资源,即没有调用closedir(fd), 如果参考该demo的人,在使用时遇到该异常时,又没有及时释放资源,可能会导致内存泄露, 另外, 第59行, 直接调用rt_free(fullpath), 还是之前的异常,如果malloc失败, fullpath为RT_NULL, 这时调用rt_free(fullpath), 应该也是异常的,需要添加释放资源判断条件
]
以下的内容不应该在提交PR时的message修改,修改下述message,PR会被直接关闭。请在提交PR后,浏览器查看PR并对以下检查项逐项check,没问题后逐条在页面上打钩。
当前拉取/合并请求的状态:
必须选择一项:
- 本拉取/合并请求是一个草稿版本
- 本拉取/合并请求是一个成熟版本
代码质量:
我在这个拉取/合并请求中已经考虑了:
- 已经仔细查看过代码改动的对比
- 代码风格正确,包括缩进空格,命名及其他风格
-
没有垃圾代码,代码尽量精简,不包含
#if 0
代码,不包含已经被注释了的代码 - 所有变更均有原因及合理的,并且不会影响到其他软件组件代码或
- 对难懂代码均提供对应的注释
- 本拉取/合并请求代码是高质量的