Change in flashrom[master]: ich_descriptors.c: Assert unsigned types >=0 on to prevent underflow ...

[Anastasia Klimchuk (Code Review)]

Attention is currently required from: Nico Huber, Angel Pons, Light.
Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/62764 )

Change subject: ich_descriptors.c: Assert unsigned types >=0 on to prevent underflow error
......................................................................

Patch Set 12:

(2 comments)

File ich_descriptors.c:

https://review.coreboot.org/c/flashrom/+/62764/comment/06fb063e_67b36cee 
PS6, Line 507: assert(j >= 12);
> If `num_regions` is 10, the second loop never runs. This doesn't change […]

Oh yes, sorry that's all good, I missed! I am marking this as resolved then?

File ich_descriptors.c:

https://review.coreboot.org/c/flashrom/+/62764/comment/0f3da6d7_81b9244a 
PS12, Line 505: ++j
Is there a reason for this change (from `j++` to `++j`) ? If not (not an intentional change), then better return back as it was. Thanks!