[riot-notifications] [RIOT-OS/RIOT] pkg/tlsf, cpu/esp_common: fix possible overflow in calloc implementations (#16438)

Marian Buschsieweke notifications at github.com
Wed May 5 09:15:30 CEST 2021


> so the newlib test breaks with current riotdocker. can we maybe drop the test change for now, backport, then merge the test fix?

Done, split out the test into a separate PR.

Being honest, I'm not sure where the two fixed calloc implementations are used; maybe they only have internal usage and the issue is impossible to trigger. But the fix is trivial, unlikely to break anything, and has only minimal impact on ROM size. IMO just getting the fix in is easier than making sure internal usage does not rely on correct calloc semantics.

-- 
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/16438#issuecomment-832467986
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210505/a7170128/attachment.htm>


More information about the notifications mailing list