Attention is currently required from: Edward O'Callaghan, Angel Pons.
Nikolai Artemiev has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/70128 )
Change subject: flashrom: Skip read/write/erase/verify ops on inaccessable flash regions ......................................................................
Patch Set 13:
(1 comment)
File flashrom.c:
https://review.coreboot.org/c/flashrom/+/70128/comment/b4b96e52_dcb8fe30 PS13, Line 348: region->name = strdup(""); This (and also strdup in ichspi) can leak memory.
Long term we should fetch the regions from the programmer once and use them to build a layout that has appropriate lifetime and cleanup handled at the level of flashrom context.
Short term we can just free region->name after every call I guess.