mirror of
https://github.com/espressif/esp-idf.git
synced 2024-09-20 10:46:02 -04:00
096c013675
...and all their callers. With the upcoming switch from sizeof(time_t)==4 to sizeof(time_t)==8, sizeof(struct stat) is also increasing. A few newlib functions present in ROM allocate 'struct stat' on the stack and call _fstat_r on this structure. The implementation of fstat is provided in ESP-IDF. This implementation will often do memset(st, 0, sizeof(*st)), where st is 'struct stat*', before setting some fields of this structure. If IDF is built with sizeof(st) different from sizeof(st) which ROM was built with, this will lead to an out-of-bounds write and a stack corruption. This commit removes problematic ROM functions from the linker script. Here are the functions which allocate 'struct stat': * _isatty_r (in ROM) * __swhatbuf_r, called by __smakebuf_r, called by __swsetup_r and __srefill_r (in ROM) * _fseeko_r (not in ROM) * glob2 (not in ROM) * _gettemp (not in ROM) As a result, these functions are used from libc.a, and use correct size of 'stat' structure. Closes https://github.com/espressif/esp-idf/issues/7980 |
||
|---|---|---|
| .. | ||
| esp32.rom.api.ld | ||
| esp32.rom.eco3.ld | ||
| esp32.rom.ld | ||
| esp32.rom.libgcc.ld | ||
| esp32.rom.newlib-data.ld | ||
| esp32.rom.newlib-funcs.ld | ||
| esp32.rom.newlib-locale.ld | ||
| esp32.rom.newlib-nano.ld | ||
| esp32.rom.newlib-time.ld | ||
| esp32.rom.redefined.ld | ||
| esp32.rom.spiflash.ld | ||
| esp32.rom.syscalls.ld | ||