7 k8 M7 Z0 f( H; D
9 P- H+ p# e. |$ P5 n/ {) N库版本V1.7.2  文件 stm32f4xx_hal_sd.c
; y. n: b: E7 a行数 503
' ^+ l$ d% b2 r1 _! A7 W0 `代码节选：
1 N! {- R6 c4 Q! k

1. HAL_StatusTypeDef HAL_SD_ReadBlocks(SD_HandleTypeDef *hsd, uint8_t *pData, uint32_t BlockAdd, uint32_t NumberOfBlocks, uint32_t Timeout)
   
2. {
   
3.   SDIO_DataInitTypeDef config;
   
4.   uint32_t errorstate = HAL_SD_ERROR_NONE;
   
5.   uint32_t tickstart = HAL_GetTick();
   
6.   uint32_t count = 0U, *tempbuff = (uint32_t *)pData;
     ~& p1 t2 \8 T1 \& T$ i5 {
7. /* ......... */
   
8. *tempbuff = SDIO_ReadFIFO(hsd->Instance);
   
9. /* ......... */
   8 V4 Q+ M% S) G; T
10. }
    

复制代码

bug1: 此处pData 为u8 指针 强转 32位 编译后使用STM指令当pData未4字节对齐时将造成非对齐访问异常。
' A$ l/ x$ S/ {& c% J" P/ zbug2: 若pData未4字节对齐，使用DMA模式时会强行向低地址进行4字节对齐造成向低地址的内存越界。
注：其中图示代码为bug1所出现的bug点，修复建议：检测对齐，发现未对齐时使用一个32位变量进行数据搬运。
  \9 u. I; }) L5 _& w. N1 Lbug2未进行查找错误点 望ST工程师自行查找。

另：对于未知的字节流数据进行非数据等长指针的强制类型转换的这种低级错误发生的ST身上令我等非常失望，希望相关人员（大学生临时工）尽快修复
" y# Y2 g% n  R! D. z! j% I, k

甩锅三连：别人怎么没问题；是你不会用；真是辣鸡

stm1024 发表于 2018-9-20 08:30
, ^# }; f& Q4 Q) U还好不是硬件bug，谈不上世界末日，而且这个锅，真的可以甩给编译器。
; ^# |+ h' n5 f* d" o) s
不过作为一个库，也确实应该考虑完 ...

不关编译器的事情 指针强转是编译器无法预判是否对齐的 所以只能用STM指令去处理

浪花雷雨 发表于 2018-9-19 17:50
认真三连问：是这样的吗？为什么这样？该改吗？

1 S" t. R0 a( A& |$ u2 ^* v回答三连：是这样的，暴露接口不对，该改

手工置顶

否认三连 ： 就是这么设计的；不是bug；下个版本再优化

认真三连问：是这样的吗？为什么这样？该改吗？

还好不是硬件bug，谈不上世界末日，而且这个锅，真的可以甩给编译器。

9 Y2 W: h* |# X$ Q不过作为一个库，也确实应该考虑完善一些。

lymzzyh 发表于 2018-9-19 17:41
甩锅三连：别人怎么没问题；是你不会用；真是辣鸡

1 @6 M$ g8 p3 B! g2 T9 z淡定一点，骚年

不用sd卡不就不会触发bug了吗，你真是辣鸡！

zero99 发表于 2018-9-20 08:45
, i/ |1 d! X7 _* i1 u$ B淡定一点，骚年

% n: b) S) c2 E3 J, D不不不 这是防喷手段 我已经帮喷子 喷过我自己了

pData实际上是R1，将R1的值复制到栈好像没啥问题...

同样遇到, 坑了一个国庆，不修天理难容

uu799 发表于 2018-10-8 14:02
同样遇到, 坑了一个国庆，不修天理难容

7 W, s, N4 T: V! G2 K3 C- B到现在都没有FAE现身说法呢

难道丢一个临时工出来背锅都不愿意的吗