search

STMicroelectronics / stm32h7b3i-eval-bsp

Provides the STM32H7B3I-EVAL board driver, part of the STM32Cube BSP Component for STM32H7 series.
BSD 3-Clause "New" or "Revised" License
2 stars 1 forks source link

static Camera_Cap pointer not initialized in stm32h7b3i_eval_camera.c #1

Open cmarsalc opened 1 year ago

cmarsalc commented 1 year ago

Describe the set-up

STM32H7B3I-EVAL STM32CubeIde 1.11.2

This is an error rather than a bug.

In https://github.com/STMicroelectronics/stm32h7b3i-eval-bsp/blob/main/stm32h7b3i_eval_camera.c the static Camera_Cap pointer is not initialized. When used it points to 0x0 so it ends up writing on ITCM RAM.

This error does not manifest as a bug in the current evaluation kit, but if you try to play with ITCM RAM it will accidentally overwrite the content, probably manifesting as a bug of some kind.

How to reproduce the bug (skip if none)

Put some data in ITC RAM address 0x0. Execute the BSP demo project.

Expected result: the ITC RAM data is intact Actual result: the ITC RAM data is corrupted by Camera_Cap pointed data writes in S5K5CAG_Probe or OV5640_Probe.

Screenshots

image

ALABSTM commented 1 year ago

Hi @cmarsalc,

Thank you for having reported. Your point looks relevant. It will be forwarded to our development teams for further analysis. We will get back to you once provided with their feedback. This may take some time. So, thank you for your comprehension and patience.

With regards,

PS: Referencing the line where the pointer is declared uninitialized.

https://github.com/STMicroelectronics/stm32h7b3i-eval-bsp/blob/1269ecd29355996b26b48fe44ea704ab56d2a8e3/stm32h7b3i_eval_camera.c#L162

TOUNSTM commented 3 months ago

ST Internal Reference: 187677