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

New C3x flashing results in "Download mismatch The system image downloaded does not match the expected checksum. Try again." #115

Closed
Deanusenvy opened this issue Mar 1, 2025 · 4 comments · Fixed by #117

Comments

@Deanusenvy
Copy link

Hi I just got my 3x after having issues with the 3 and trying all different ports and cables I keep getting "Download mismatch
The system image downloaded does not match the expected checksum. Try again." nelsonjchen asked me to open an issue here in the discord convo. He said "There was a slightly hasty change made last night In retrospect, I didn't know it went in that fast"
Thanks, Dean

Image

@adeebshihadeh
Copy link
Collaborator

Reverted, @incognitojam is flash not compatible with the new manifest?

@Deanusenvy
Copy link
Author

Deanusenvy commented Mar 1, 2025 via email

@maxime-desroches
Copy link

@incognitojam Since we are now including sparse images in the manifest, the hash_raw field is a bit different for those: commaai/agnos-builder#443 (comment)

@incognitojam
Copy link
Member

I think the hash stuff was changed after I reviewed it 😕 And it wasn't checked automatically in Flash CI because the download/unpack is slow (> 10 mins on the actions runner), so I skipped the tests for the system image in CI and only checked it manually with the earlier manifest. #116 will help to highlight this in future with a slow but non-blocking CI job.

The cause of this issue is that alt.hash is like a hash_raw instead of a SHA-256 checksum of the image. #117 should fix this by ignoring the alt image.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants