[flashrom-gerrit] [S] Change in flashrom[master]: erasure_layout: Fix double invocation of erasers

[Aarya (Code Review)]

Attention is currently required from: Alexander Goncharov, Anastasia Klimchuk, Sam McNally.

Aarya has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/77747?usp=email )

Change subject: erasure_layout: Fix double invocation of erasers
......................................................................

Patch Set 3:

(1 comment)

Patchset:

PS2:
> Also to add here: I actually have never seen the condition being called `if (last_addr < start_addr  […]

Yes that condition wouldn't be called, because iirc the erase addresses are extend d to match the block boundaries. So ideally there should nothing be left once the above  loop at line 309 finishes.

also I'm not sure if understand the double eraser invocation correctly, is it that a block is getting erased twice?