[riot-notifications] [RIOT-OS/RIOT] nanocoap: bugfix token overflow (#14075)

Ken Bannister notifications at github.com
Mon May 25 15:14:25 CEST 2020


@kb2ma commented on this pull request.

Nice work, @mjurczak! I would not change a thing. Thanks for the unit test. I also tested all of the failure conditions, and it all works as expected.

Please squash the first three commits down to one for the pointer range check, and then keep the token length commit as is.



-- 
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/14075#pullrequestreview-417702678
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200525/178f2ca9/attachment.htm>


More information about the notifications mailing list