-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Lora: Fix LoRaMacCrypto asserts #6800
Conversation
If Lora stack is built with incorrect mbedtls settings, crypto methods should assert and return error value. This fixes MBED_ASSERTs to work correctly.
@hasnainvirk @kjbracey-arm @AnttiKauppila please review. |
jenkins CI - started failing yesterday afternoon, we are investigating the cause, might not be related to this changeset. |
I restarted jenkins CI, should be green if all OK here. The CI issue was resolved. |
/morph build |
Build : SUCCESSBuild number : 1932 Triggering tests/morph test |
Test : SUCCESSBuild number : 1754 |
Exporter Build : SUCCESSBuild number : 1578 |
This fix is on top of #6692 which is targeted for 5.9. |
Description
If Lora stack is built with incorrect mbedtls settings, crypto methods
should assert and return error value. This fixes MBED_ASSERTs to
work correctly.
This is a fix for issue #6764.
Pull request type