Presence detection can integer-overflow on baseline + threshold (u16)
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 76/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Tranquilla
- Stack tecnologico
- rust
- Ambito
- embedded-iot
Direzione di ricerca
Inizia in crates/podd-core/src/sensor/presence.rs, nel confronto di update_presence, poi leggi come viene analizzato config.threshold in config/mqtt.rs. L’issue identifica l’overflow e suggerisce un’addizione saturante o di ampliare il confronto; controlla i test esistenti del progetto per il rilevamento della presenza e aggiungi una verifica per un valore di base più una soglia superiore a 65535. Il lavoro è completo quando il confronto evita l’overflow nelle build debug e release e il caso di regressione passa.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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).
- Lingua principale
- Rust
- Stelle
- 0
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di GoogleBot42/podd
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 84/100
GoogleBot42/podd#63 ·
-
documentation
Difficoltà 1/5 1-3 ore Idoneità per principianti 78/100
GoogleBot42/podd#62 ·
-
documentation good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 69/100
GoogleBot42/podd#60 ·
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
GoogleBot42/podd#61 ·
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
GoogleBot42/podd#58 ·
Tutte le issue di GoogleBot42/podd
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
rubys/roundhouse#1917 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 2 giorni
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
[cosmic-applet-battery] Division by zero displays NaN% when UPower reports max_brightness = 0Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
pop-os/cosmic-applets#1619 ·
I maintainer di solito rispondono entro 3 giorni
-
binding:python bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno