Re: [U-Boot] [PATCH] common: env_flags: fix loop condition when using env_flags_varaccess_mask

13 Jan 2016


      On Tue, Jan 12, 2016 at 3:23 AM, Peng Fan van.freenix@gmail.com wrote:
...
From: Peng Fan peng.fan@nxp.com
We should use ARRAY_SIZE, but not sizeof. The size of
env_flags_varaccess_mask is 16bytes, but we only need 4 loops.
If using 16 as the end condition, we may access memory that
not belong to array env_flags_varaccess_mask.
Signed-off-by: Peng Fan peng.fan@nxp.com
Cc: Joe Hershberger joe.hershberger@ni.com
Cc: York Sun yorksun@freescale.com
Cc: Simon Glass sjg@chromium.org
Cc: Tom Rini trini@konsulko.com

Acked-by: Joe Hershberger joe.hershberger@ni.com