codec command() silently truncates any payload longer than 255 bytes into a corrupt frame

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#41 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
76/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Ruhig
Tech-Stack
rust
Bereich
backend-api-design

Rechercherichtung

Lies crates/pod-proto/src/codec.rs im Bereich von Zeile 104, wo command() die Payload-Länge in ein Byte umwandelt. Prüfe, wie die Funktion Fehler meldet und wie ihre Aufrufer ungültige Payloads behandeln, und füge dann die angeforderte Längenprüfung hinzu. Fertig ist die Arbeit, wenn eine Payload mit mehr als 255 Byte abgewiesen wird oder die vorgesehene Debug-Assertion auslöst und gültige Frames ihr bisheriges Verhalten beibehalten.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug

codec.rs command() writes the length byte as res.push(payload.len() as u8) (crates/pod-proto/src/codec.rs:104). A payload of 256+ bytes wraps mod 256, emitting a LEN that disagrees with the checksum (computed over the full payload), so the device sees a malformed/short frame. No command builds a >255B payload today (SetAlarm is the largest at 8B), and only SensorCommand::Random(Vec) is unbounded, so it is not currently triggered. But the LSP length field is inherently one byte, and the not-yet-built live MCU flash path (TODO(live-cutover)) will need to stream firmware — if it ever sends chunks through command(), oversized chunks would produce corrupt frames to an MCU mid-flash. Add a debug_assert/hard error when payload.len() > 255.

Filed from a repo-wide code audit (working tree at bcad746).

Vorherrschende Sprache
Rust
Sterne
0
Forks
0
Keine gemergten PRs in 30 T.

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus GoogleBot42/podd

Alle Issues in GoogleBot42/podd

Ähnliche Issues

Weitere Issues zu Rust

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.