Skip to content

Conversation

@ioannisg
Copy link
Member

This commit fixes a bug in _get_region_ap(.) by adding
parentheses to enforce the correct operator precedence.

Fixes #8864.

Signed-off-by: Ioannis Glaropoulos Ioannis.Glaropoulos@nordicsemi.no

This commit fixes a bug in _get_region_ap(.) by adding
parentheses to enforce the correct operator precedence.

Fixes zephyrproject-rtos#8864.

Signed-off-by: Ioannis Glaropoulos <Ioannis.Glaropoulos@nordicsemi.no>
@ioannisg ioannisg added area: ARM ARM (32-bit) Architecture area: Memory Protection labels Jul 12, 2018
@ioannisg ioannisg added the bug The issue is a bug, or the PR is fixing a bug label Jul 12, 2018
@ioannisg
Copy link
Member Author

Bug was, unfortunately, introduced while doing the ARM MPU refactor work.

@codecov-io
Copy link

Codecov Report

Merging #8868 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #8868   +/-   ##
=======================================
  Coverage   52.31%   52.31%           
=======================================
  Files         195      195           
  Lines       24732    24732           
  Branches     5141     5141           
=======================================
  Hits        12939    12939           
  Misses       9719     9719           
  Partials     2074     2074

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6b836d6...c2788ee. Read the comment docs.

@ioannisg ioannisg added the priority: high High impact/importance bug label Jul 12, 2018
@MaureenHelm MaureenHelm merged commit 8bfddb5 into zephyrproject-rtos:master Jul 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ARM ARM (32-bit) Architecture area: Memory Protection bug The issue is a bug, or the PR is fixing a bug priority: high High impact/importance bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ARM MPU _arch_buffer_validate allowing reads to kernel memory

4 participants