diff options
| author | Peng Fan <peng.fan@nxp.com> | 2016-01-12 13:21:23 +0800 |
|---|---|---|
| committer | guoyin.chen <guoyin.chen@freescale.com> | 2016-03-04 15:53:39 +0800 |
| commit | 420788d38f9d9ff1db38cccc42d415211bd8ddfb (patch) | |
| tree | 2a668062befa31a4d39c2b1fa858a52563403890 | |
| parent | 72f69ee87cb88d4e18fe6b05701bff275dc1b68f (diff) | |
| download | u-boot-imx-420788d38f9d9ff1db38cccc42d415211bd8ddfb.zip u-boot-imx-420788d38f9d9ff1db38cccc42d415211bd8ddfb.tar.gz u-boot-imx-420788d38f9d9ff1db38cccc42d415211bd8ddfb.tar.bz2 | |
MLK-12201 common: env: fix out of bounds read
We should use ARRAY_SIZE, but not directly sizeof, otherwise
we may access memory that is not belong the array env_flags_varaccess_mask.
Coverity ID: 17949
Signed-off-by: Peng Fan <peng.fan@nxp.com>
| -rw-r--r-- | common/env_flags.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/common/env_flags.c b/common/env_flags.c index 471973a..8e60384 100644 --- a/common/env_flags.c +++ b/common/env_flags.c @@ -152,7 +152,7 @@ enum env_flags_varaccess env_flags_parse_varaccess_from_binflags(int binflags) { int i; - for (i = 0; i < sizeof(env_flags_varaccess_mask); i++) + for (i = 0; i < ARRAY_SIZE(env_flags_varaccess_mask); i++) if (env_flags_varaccess_mask[i] == (binflags & ENV_FLAGS_VARACCESS_BIN_MASK)) return (enum env_flags_varaccess)i; |