战舰实验Flash模拟EEPROM有个小隐患~

keyearth

void STMFLASH_Write(u32 WriteAddr,u16 *pBuffer,u16 NumToWrite) {          if(WriteAddr<STM32_FLASH_BASE||(WriteAddr>=(STM32_FLASH_BASE+1024*STM32_FLASH_SIZE)))return;//非法地址

这个判断是否是非法地址的语句少判断了两种非法情况，一个是地址是否为偶数，一个是所有要写入的数据地址是否超出。
我改成下面这样了：

if( (WriteAddr < STM32_FLASH_BASE) || (WriteAddr%2)

||((WriteAddr+2*NumToWrite) >= (STM32_FLASH_BASE+1024*STM32_FLASH_SIZE)))

return;//非法地址

供大家参考。

ST-FAE-SALE

不错，可以参考一下

lanny_cn

原子大多例程都对参数没做严格检查的，也有好处：不要浪费CPU时间，有不好：调试的时候麻烦会多些

可以参考usosii里的，比如定义个_DEBUG_>0，然后就做参数检查，否则就不做检查

正点原子

回复【楼主位】keyearth:
---------------------------------
谢谢指出.

MrXiong

谢谢楼主，刚好在学习中，这个帖正好给予很好的帮助！