Presence detection can integer-overflow on baseline + threshold (u16)
还没有人认领这个 Issue。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 76/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 冷清
- 技术栈
- rust
- 领域
- embedded-iot
调研方向
从 crates/podd-core/src/sensor/presence.rs 中 update_presence 的比较开始，然后查看 config/mqtt.rs 中如何解析 config.threshold。Issue 指出了溢出问题，并建议使用饱和加法或扩大比较的数值类型；检查项目中现有的存在检测测试，并补充基线值加上高于 65535 的阈值的测试。完成标准是比较在 debug 和 release 构建中都不会溢出，且回归用例通过。
由索引模型根据 Issue 内容生成。
描述
update_presence tests data.values[i] > config.baselines[i] + config.threshold with both operands u16 (crates/podd-core/src/sensor/presence.rs:80). Baselines are the averaged raw capacitance readings and threshold is user-settable (parsed as a full u16 in config/mqtt.rs:270), so baseline+threshold can exceed 65535. In a release build this wraps to a small number, making the side read as present continuously; in a debug build it panics. Use saturating_add or widen to u32 for the comparison.
Filed from a repo-wide code audit (working tree at bcad746).
- 主要语言
- Rust
- 星标
- 0
- 派生
- 0
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
GoogleBot42/podd 的其他 Issue
-
documentation
难度 1/5 1 小时以内 新手友好度 84/100
GoogleBot42/podd#63 ·
-
documentation
难度 1/5 1-3 小时 新手友好度 78/100
GoogleBot42/podd#62 ·
-
documentation good first issue
难度 2/5 1-3 小时 新手友好度 69/100
GoogleBot42/podd#60 ·
-
documentation
难度 1/5 1 小时以内 新手友好度 85/100
GoogleBot42/podd#61 ·
-
enhancement
难度 2/5 1-3 小时 新手友好度 72/100
GoogleBot42/podd#58 ·
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 76/100
rubys/roundhouse#1917 ·
-
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 2 天内回复
-
good first issue
难度 2/5 1-3 小时 新手友好度 88/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 84/100
pop-os/cosmic-applets#1619 ·
维护者通常 3 天内回复
-
binding:python bug
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复