michaelh/zephyr
1
0
Fork 0
Code Releases Activity

arch: arm: mpu: Fix -Wstringop-overread warning

Browse source 
GCC 12 performs bounds checking on the pointer arguments specified like
an array (e.g. `int arg[]`) and treats such arguments with an empty
length as having the length of 0, resulting in the compiler printing
out `stringop-overread' warning when they are accessed.

This commit corrects any pointer arguments declared using the array
expression to use the pointer expression instead.

Signed-off-by: Stephanos Ioannidis <root@stephanos.io>
This commit is contained in:
Stephanos Ioannidis 2022-06-15 22:08:07 +09:00
commit 8506979f27
2 changed files with 9 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

14
arch/arm/core/aarch32/mpu/arm_core_mpu_dev.h
View file

@ -118,8 +118,8 @@ struct k_thread;
*
* The function shall be invoked once, upon system initialization.
*
* @param static_regions[] an array of pointers to memory partitions
* to be programmed
* @param static_regions an array of pointers to memory partitions
* to be programmed
* @param regions_num the number of regions to be programmed
* @param background_area_start the start address of the background memory area
* @param background_area_end the end address of the background memory area
@ -132,7 +132,7 @@ struct k_thread;
* requirements of the MPU hardware.
*/
void arm_core_mpu_configure_static_mpu_regions(
const struct z_arm_mpu_partition static_regions[],
const struct z_arm_mpu_partition *static_regions,
const uint8_t regions_num, const uint32_t background_area_start,
const uint32_t background_area_end);
@ -164,7 +164,7 @@ void arm_core_mpu_configure_static_mpu_regions(
* arm_core_mpu_configure_static_mpu_regions().
*/
void arm_core_mpu_mark_areas_for_dynamic_regions(
const struct z_arm_mpu_partition dyn_region_areas[],
const struct z_arm_mpu_partition *dyn_region_areas,
const uint8_t dyn_region_areas_num);
#endif /* CONFIG_MPU_REQUIRES_NON_OVERLAPPING_REGIONS */
@ -176,8 +176,8 @@ void arm_core_mpu_mark_areas_for_dynamic_regions(
* within a (background) memory area. The total number of HW MPU regions
* to be programmed depends on the MPU architecture.
*
* @param dynamic_regions[] an array of pointers to memory partitions
* to be programmed
* @param dynamic_regions an array of pointers to memory partitions
* to be programmed
* @param regions_num the number of regions to be programmed
*
* The function shall assert if the operation cannot be not performed
@ -185,7 +185,7 @@ void arm_core_mpu_mark_areas_for_dynamic_regions(
* not exceed the number of (currently) available MPU indices.
*/
void arm_core_mpu_configure_dynamic_mpu_regions(
const struct z_arm_mpu_partition dynamic_regions[],
const struct z_arm_mpu_partition *dynamic_regions,
uint8_t regions_num);
#if defined(CONFIG_USERSPACE)

4
arch/arm/core/aarch32/mpu/arm_mpu.c
View file

@ -269,7 +269,7 @@ int arm_core_mpu_buffer_validate(void *addr, size_t size, int write)
* @brief configure fixed (static) MPU regions.
*/
void arm_core_mpu_configure_static_mpu_regions(const struct z_arm_mpu_partition
static_regions[], const uint8_t regions_num,
*static_regions, const uint8_t regions_num,
const uint32_t background_area_start, const uint32_t background_area_end)
{
if (mpu_configure_static_mpu_regions(static_regions, regions_num,
@ -301,7 +301,7 @@ void arm_core_mpu_mark_areas_for_dynamic_regions(
* @brief configure dynamic MPU regions.
*/
void arm_core_mpu_configure_dynamic_mpu_regions(const struct z_arm_mpu_partition
dynamic_regions[], uint8_t regions_num)
*dynamic_regions, uint8_t regions_num)
{
if (mpu_configure_dynamic_mpu_regions(dynamic_regions, regions_num)
== -EINVAL) {