[riot-notifications] [RIOT-OS/RIOT] tests-crypto-modes-ccm: Refactoring to support more evolved tests (#8109)

Gaëtan Harter notifications at github.com
Tue Dec 19 17:54:57 CET 2017


cladmi commented on this pull request.



> @@ -17,6 +17,8 @@
 #include "crypto/modes/ccm.h"
 #include "tests-crypto.h"
 
+static const uint8_t nonce_and_len_encoding_size = 15;

I based myself of the ccm_encrypt and decrypt api:

https://github.com/RIOT-OS/RIOT/blob/b428979a1da99e4dac42688f6d3e61f7218176ed/sys/include/crypto/modes/ccm.h#L54-L57

* `mac_length` and `length_encoding` are both `uint8_t`.
* `nonce_len` which is a `size_t` has a maximum length of `15 - length_encoding` (the api documentation is wrong and saying 16 - L, I will do a PR).

Whereas the other ones `input_len`, `output_len` and `auth_data_len` can be bigger than 255 and its required to provide big data to trigger some of the bugs.


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/8109#discussion_r157812880
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20171219/2bc34921/attachment.html>


More information about the notifications mailing list