michaelh/zephyr
1
0
Fork 0
Code Releases Activity

arc: fix off-by-one in _is_in_region()

Browse source 
Similar issue to what was fixed earlier in the MPUv3
code. start + size should be <= r_addr_end. Fixes
a problem where the last byte of an MPU region is
incorrectly reported as out-of-bounds.

Fixes: #15131

Signed-off-by: Andrew Boie <andrew.p.boie@intel.com>
This commit is contained in:
Andrew Boie 2019-04-02 18:04:14 -07:00 • committed by Anas Nashif
commit 29095ac135
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
arch/arc/core/mpu/arc_mpu_v2_internal.h
View file

@ -118,7 +118,7 @@ static inline bool _is_in_region(u32_t r_index, u32_t start, u32_t size)
r_size_lshift = (r_size_lshift & 0x3) | ((r_size_lshift >> 7) & 0x1C);
r_addr_end = r_addr_start + (1 << (r_size_lshift + 1));
if (start >= r_addr_start && (start + size) < r_addr_end) {
if (start >= r_addr_start && (start + size) <= r_addr_end) {
return 1;
}