nuttx/nuttx-update
1
2
Fork 1
Code Issues 1 Pull requests 1 Projects Releases Packages Wiki Activity Actions

samv7: fix bytes to words calculation in user signature read

Browse source 
EEFC read sequence requires read length in words instead of bytes,
therefore bytes given by the user has to be recalculated to words.
The calculation however had a mistake in brackets and was just adding
1 to buflen instead of recalculating it to 4 byte words. This caused
global array g_page_buffer to overflow for some reads.

This fixes the calculation.

Signed-off-by: Michal Lenc <michallenc@seznam.cz>
This commit is contained in:
Michal Lenc 2024-12-02 10:52:29 +01:00 committed by Xiang Xiao
parent 140b3080c5
commit 9fbb81e8a4
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
arch/arm/src/samv7/sam_us.c
View file

@ -177,7 +177,7 @@ int sam_read_user_signature(void *buffer, size_t buflen)
/* sam_eefc_readsequence requires read length in bit words. */
nwords = (buflen + sizeof(uint32_t) / sizeof(uint32_t));
nwords = (buflen + sizeof(uint32_t)) / sizeof(uint32_t);
sam_eefc_readsequence(FCMD_STUS, FCMD_SPUS, g_page_buffer, nwords);
/* Copy local buffer to void *buffer provided by the user. */