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:35:54 +0800 |
| commit | d72087c428ee44dcfe527d4eff4270ad51a8f917 (patch) | |
| tree | 2a668062befa31a4d39c2b1fa858a52563403890 | |
| parent | eb8b3f27e2937d8856fe01441b6c2880b59f02b5 (diff) | |
| download | u-boot-imx-d72087c428ee44dcfe527d4eff4270ad51a8f917.zip u-boot-imx-d72087c428ee44dcfe527d4eff4270ad51a8f917.tar.gz u-boot-imx-d72087c428ee44dcfe527d4eff4270ad51a8f917.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; |