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

Enable CCM demo to work on nrf51 #223

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

therealprof
Copy link
Contributor

Signed-off-by: Daniel Egger daniel@eggers-club.de

Signed-off-by: Daniel Egger <daniel@eggers-club.de>
@thalesfragoso
Copy link
Contributor

nRF51 devices only support payloads of up to 27 bytes.

Maybe we should change the example to use smaller payloads, I remember that I used those sizes because I wanted to try some bench-marking. Since I already included the example as a "benchmark" (see stopwatch.rs) maybe we should use some cfgs also on the buffer size.

@therealprof
Copy link
Contributor Author

Huh? Where does that quoted comment come from? 😅

@thalesfragoso
Copy link
Contributor

From the docs:

//! * nRF51 devices only support payloads of up to 27 bytes.

And from the manual:

Encryption (Page 121):

The AES CCM is limited to read maximum 27 bytes of the unencrypted payload (PL) regardless of what is specified in
the length field of the unencrypted packet.

Decryption (Page 122):

The AES CCM is limited to read maximum 27 bytes of the encrypted payload and four bytes of the MIC regardless of
what is specified in the length field of the encrypted packet.

@therealprof therealprof marked this pull request as draft September 18, 2020 07:00
@therealprof
Copy link
Contributor Author

I'll modify the example to use less payload on nrf51 when I find time (or someone beats me to it 😉 ).

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 this pull request may close these issues.

None yet

2 participants