[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:
https://github.com/RIOT-OS/RIOT/pull/13824#discussion_r427821901
-------------- 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