Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

safety: check interceptor msg counter #1738

Merged
merged 4 commits into from
Nov 22, 2023
Merged

Conversation

sshane
Copy link
Contributor

@sshane sshane commented Nov 22, 2023

Simpler version of #1735, solves mismatch caught in commaai/openpilot#30443

@sshane sshane marked this pull request as ready for review November 22, 2023 01:19
@sshane sshane merged commit 83acc65 into master Nov 22, 2023
10 of 11 checks passed
@sshane sshane deleted the interceptor-check-counter branch November 22, 2023 01:23
@sshane sshane added the car safety vehicle-specific safety code label Nov 22, 2023
adeebshihadeh added a commit that referenced this pull request Nov 25, 2023
adeebshihadeh added a commit that referenced this pull request Nov 25, 2023
@sshane sshane restored the interceptor-check-counter branch December 9, 2023 13:20
@sshane sshane deleted the interceptor-check-counter branch December 9, 2023 13:33
sshane added a commit that referenced this pull request Dec 14, 2023
* toyota: check interceptor counter

* add counters and check rx

* honda: check counter

* clean up
sshane added a commit that referenced this pull request Dec 14, 2023
* safety: check interceptor msg counter (#1738)

* toyota: check interceptor counter

* add counters and check rx

* honda: check counter

* clean up

* revert

* typing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
car safety vehicle-specific safety code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant