Is the check avail >= SIMPLEFS_MAX_EXTENTS actually necessary here?
还没有人认领这个 Issue。
评估
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 新手友好度
- 48/100
- Issue 类型
- 重构
- 描述清晰度
- 基本清楚
- 活跃度
- 冷清
- 技术栈
- c, linux
调研方向
Inspect simplefs_create and simplefs_get_available_ext_idx, then trace how eblock->nr_files and avail are updated across the relevant directory operations. Determine whether SIMPLEFS_MAX_EXTENTS can be exceeded despite the existing SIMPLEFS_MAX_SUBFILES check; done means reaching a justified decision about retaining or removing the boundary check and validating the affected filesystem behavior.
由索引模型根据 Issue 内容生成。
描述
The purpose of this check is to prevent avail from going out of bounds, but I think it may not be necessary.
/* Validate avail index is within bounds */
if (avail >= SIMPLEFS_MAX_EXTENTS) {
ret = -EMLINK;
goto iput;
}
Whenever functions like "simplefs_create" are called, we already validate the maximum number of files with:
if (eblock->nr_files == SIMPLEFS_MAX_SUBFILES)
If this check passes, it means there is guaranteed to be space for creating a new object under this directory. Therefore, simplefs_get_available_ext_idx() should never return an out-of-bound value, and this boundary check appears redundant.
- 主要语言
- C
- 星标
- 495
- 派生
- 110
- 平均合并
- 26 分钟
- 30 天内合并 PR
- 1
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
sysprog21/simplefs 的其他 Issue
-
Implement journaling support可能重新可做 @HotMercury 于 891 天前认领，目前没有进行中的 PR。 未关闭
查看 sysprog21/simplefs 的全部 Issue
相似的 Issue
-
[BUG] Small issues on iMXRT found by Claude可能已有人在做 @Sarthak-Pandey 今天认领。 未关闭Arch: arm Area: Networking OS: Linux Type: Bug
难度 2/5 1-3 小时 新手友好度 66/100
维护者通常 1 天内回复
-
[Issue]: hipsparselt.h C API as shipped now requires C23 -- ONLY because of missing include or guard未关闭
难度 2/5 1-3 小时 新手友好度 62/100
ROCm/rocm-libraries#13736 ·
维护者通常 2 天内回复
-
难度 2/5 1-3 小时 新手友好度 80/100
维护者通常 1 天内回复
-
`inverse()` of a controlled `CUGate` drops the phase parameter γ可能已有人在做 @rupasri118 今天认领。 未关闭bug mod: circuit
难度 2/5 1-3 小时 新手友好度 66/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
PackageKit/PackageKit#1026 ·