[riot-notifications] [RIOT-OS/RIOT] pkg/openwsn: re-integrate the network stack as a package (#13824)

Francisco notifications at github.com
Wed May 20 10:13:46 CEST 2020

@fjmolinas commented on this pull request.

> @@ -42,6 +42,10 @@ extern "C" {
 #define CCM_BLOCK_SIZE                      16
+ * @brief Maximum length for the appended MAC
+ */
+#define CCM_MAC_MAX_LEN                     16

It was not needed before, but @kaspar030 had made a comment about avoiding VLA, and the max size for the MAC in CCM is 16 bytes, I preferred adding a define to the header than defining it in the openwsn code.

I could add it in its own PR, but it would have no use case, same for d924a3ff0d41344209287ae2401da9ec4bc56b5a

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200520/84c30f5f/attachment.htm>

More information about the notifications mailing list