platformio / platform-espressif8266

Espressif 8266: development platform for PlatformIO
https://registry.platformio.org/platforms/platformio/espressif8266
Apache License 2.0
321 stars 218 forks source link

tool-mklittlefs is outdated - cannot handle paths > 32 bytes #241

Closed minimum-necessary-change closed 3 years ago

minimum-necessary-change commented 3 years ago

fixed at - https://github.com/earlephilhower/mklittlefs/commit/0beba0fe0f0396f9690139930e338e448cd6507d

When building a filesystem image and a path greater that 32 bytes is encountered will result in a crash

Building in release mode
"mklittlefs" -c data -p 256 -b 8192 -s 3121152 .pio/build/nodemcuv2/littlefs.bin
/web/index.html.gz
*** buffer overflow detected ***: terminated
/web/src/css/bootstrap.min.css.gz
Aborted (core dumped)

in the example above the path is "/web/src/css/bootstrap.min.css.gz" equaling 34 bytes.

$ pio update --dry-run
...
Platform espressif8266
--------
Checking platformio/espressif8266             2.6.2                              [Up-to-date]
Checking platformio/toolchain-xtensa          2.40802.200502 @ ~2.40802.191122   [Up-to-date]
Checking platformio/framework-arduinoespressif8266 3.20704.0 @ ~3.20704.0             [Up-to-date]
Checking platformio/tool-esptool              1.413.0 @ <2                       [Up-to-date]
Checking platformio/tool-esptoolpy            1.20800.0 @ ~1.20800.0             [Up-to-date]
Checking platformio/tool-mkspiffs             1.200.0 @ ~1.200.0                 [Up-to-date]
Checking platformio/tool-mklittlefs           1.203.200522 @ ~1.203.0            [Up-to-date]
valeros commented 3 years ago

The latest version of the tool-mklittlefs package has been uploaded to the registry, please update via pio platform update espressif8266

minimum-necessary-change commented 3 years ago

Thanks!