Messages in this thread | | | Date | Mon, 11 Nov 2019 13:05:16 +0000 | From | Chris Down <> | Subject | Re: [PATCH -next] mm/vmscan: fix an undefined behavior for zone id |
| |
Qian Cai writes: >> On Nov 8, 2019, at 3:44 PM, Qian Cai <cai@lca.pw> wrote: >> >> - for (zid = 0; zid <= zone_idx; zid++) { >> + for (zid = 0; zid < zone_idx; zid++) { >> struct zone *zone = > >Oops, I think here needs to be, > >for (zid = 0; zid <= zone_idx && zid < MAX_NR_ZONES; zid++) { > >to deal with this MAX_NR_ZONES special case.
Ah, I just saw this in my local checkout and thought it was from my changes, until I saw it's also on clean mmots checkout. Thanks for the fixup!
Acked-by: Chris Down <chris@chrisdown.name>
| |