[riot-notifications] [RIOT-OS/RIOT] tests/unittests/tests-ipv6_hdr: fix too short ipv6_hdr_t allocations (#16616)

Francisco notifications at github.com
Tue Jul 6 10:06:50 CEST 2021


@fjmolinas approved this pull request.

ACK, I guess gcc didn't complain before because the functions only touched the available bytes, but it seems reasonable to allocate the right amount. The only downside I see is a bit of more RAM in the tests, but I think its better to be explicit here and its a test so we can handle the extra RAM. L'est wait a bit to see if @miri64 has a comment before merging but otherwise lets go with it.



-- 
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/16616#pullrequestreview-699655044
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210706/067cd8e7/attachment.htm>


More information about the notifications mailing list