[coreboot-gerrit] [S] Change in coreboot[master]: cbfstool: Correctly interpret uninitialized MMAP window ranges

[Julius Werner (Code Review)]

Attention is currently required from: Arthur Heymans, Kyösti Mälkki, Tim Wawrzynczak.

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/65417?usp=email )

Change subject: cbfstool: Correctly interpret uninitialized MMAP window ranges
......................................................................

Patch Set 1:

(1 comment)

Patchset:

PS1:
> Was the unresolved comment solved already?

I don't have any memory of this patch. Arthur has since changed a bunch of how this works in CB:68160. I believe(?) that with his patch it shouldn't be possible to have a mapping window of size 0 anymore (although I'm not quite sure how it was previously possible to begin with). I think this patch is probably still valid as a sanity check but I don't think it should be possible to hit that case anymore.