_sbrk has no bounds checking

[WestfW]

_sbrk has no bounds checking

[matthijskooijman]

I recently ran into this for samd too, which I think is similar to sam. I have a local fix I should create a pullrequest for, but got distracted with other things before I could finish it. I'll try to see if I can push it out maybe next week, though I probably won't have time to port it to sam as well.

[matthijskooijman]

I just published my fix for samd at arduino/ArduinoCore-samd#681, see that PR for a more detailed analysis (also check the commit messages) and a test sketch.

Looking at the sam core, it seems that the problem is essentially the same (sbrk() never failing), but where samd uses gcc's libnosys to provide it, the sam core (like @WestfW already mentioned) just contains a broken _sbrk() itself:

ArduinoCore-sam/cores/arduino/syscalls_sam3.c

Line 64 in 790ff2c

extern caddr_t _sbrk ( int incr )

The fix is the same, though, just add a proper _sbrk() implementation. At first glance the samd version from my PR would be suitable as-is, except samd uses end and sam uses _end to signal the end of global variables.

I won't have time to prepare a PR for this, but would be happy to review and assist if anyone else does so.