commaai / commaai/opendbc

IsoTpMessage: more sanity checks

Open
#1,685 0 comments 0 reactions 0 assignees View on GitHub
bug enhancement
Dominant language
Python
Stars
3.4k
Forks
2.3k
Avg merge
3d 2h
Merged PRs (30d)
38

Description

We should assert we can only ever receive one single or one first frames, in case we are operating with an rx address that is actually a broadcast address and the data *kind* of looks correct. We have a similar check for consecutive frames to make sure we're in a good state to receive them.

Related to https://github.com/commaai/opendbc/issues/1684

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.