[riot-notifications] [RIOT-OS/RIOT] uhcp: fix handling of pushed prefix (#15359)

Kaspar Schleiser notifications at github.com
Mon Nov 2 15:39:11 CET 2020


@kaspar030 commented on this pull request.



> @@ -97,14 +97,17 @@ void uhcp_handle_push(uhcp_push_t *req, uint8_t *src, uint16_t port, uhcp_iface_
     char addr_str[INET6_ADDRSTRLEN];
     char prefix_str[INET6_ADDRSTRLEN];
     inet_ntop(AF_INET6, src, addr_str, INET6_ADDRSTRLEN);
-    uint8_t prefix[16];
+    uint8_t prefix[IN6ADDRSZ] = { 0 };

this is an implicit call to memset, maybe move below the sanity check?

-- 
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/15359#pullrequestreview-521686433
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20201102/823414f4/attachment.htm>


More information about the notifications mailing list