关于指针的使用 指针, 时间, 仿真器, 消息

jinandawei

我们的项目使用CAN接口进行数据的传输，通过中断接收CAN消息。主函数中完成接收CAN消息后的处理。前两天在我们的项目中出现问题：
主函数不能正常运行，通过仿真器看到好像程序大多的时间都在CAN收中断中忙活。

于是乎，就开始找中断的问题，本来以为是中断函数中添加了太多的处理所致。下面是我的代码：

1. 
   
2. //滤除接受CAN消息中的杂波
   
3. //入口参数：PCAN_MSG pcan_msg_dst CAN消息结构体数组的首地址，PCAN_MSG pcan_msg_src 接收到的CAN消息地址
   
4. void FiltReceivdCanMsg(PCAN_MSG pcan_msg_dst,PCAN_MSG pcan_msg_src)
   
5. {
   
6. byte i=0;
   
7. if((pcan_msg_dst[0].data[0]^pcan_msg_src->data[0])||(pcan_msg_dst[0].data[1]^pcan_msg_src->data[1])||
   
8. (pcan_msg_dst[0].data[2]^pcan_msg_src->data[2])||(pcan_msg_dst[0].data[3]^pcan_msg_src->data[3])||
   
9. (pcan_msg_dst[0].data[4]^pcan_msg_src->data[4])||(pcan_msg_dst[0].data[5]^pcan_msg_src->data[5])||
   
10. (pcan_msg_dst[0].data[6]^pcan_msg_src->data[6])||(pcan_msg_dst[0].data[7]^pcan_msg_src->data[7])) //不相等
    
11. {
    
12. for(i=0;i<CAN_LENTH;i++)
    
13. {
    
14. pcan_msg_dst[0].data=pcan_msg_src->data;
    
15. }
    
16. }else
    
17. {
    
18. for(i=0;i<CAN_LENTH;i++)
    
19. {
    
20. pcan_msg_dst[1].data=pcan_msg_src->data;
    
21. }
    
22. }
    
23. }
    
24. 
    
25. //复制数据到相应的CAN消息缓存区
    
26. void CopyDataToRAM(PCAN_MSG pcan_msg)
    
27. {
    
28. dword id;
    
29. PCAN_MSG pcan_msg_dst;
    
30. if (IS_BIT_SET(pcan_msg->id.b[1],4)&&IS_BIT_SET(pcan_msg->id.b[1],3)&&IS_BIT_CLR(pcan_msg->id.b[3],0))//ID满足扩展数据帧的条件
    
31. {
    
32. id=GetPureID(pcan_msg);
    
33. if((id==BCM1_STATUS1_ID)||(id==BCM1_STATUS2_ID)||(id==BCM1_STATUS3_ID)||(id==BCM1_DIAG1_ID)||
    
34. (id==BCM1_INFO1_ID)||(id==DashBoard_INFO_ID))
    
35. {
    
36. if(id==BCM1_STATUS1_ID)
    
37. {
    
38. CANMSG_BCM1_STATUS1[1].id.dw=id;
    
39. pcan_msg_dst=CANMSG_BCM1_STATUS1;//复制驾驶室输入状态信息1的CAN消息
    
40. }else if(id==BCM1_STATUS2_ID)
    
41. {
    
42. CANMSG_BCM1_STATUS2[1].id.dw=id;
    
43. pcan_msg_dst=CANMSG_BCM1_STATUS2;//复制驾驶室输入状态信息2的CAN消息
    
44. }else if(id==BCM1_STATUS3_ID)
    
45. {
    
46. CANMSG_BCM1_STATUS3[1].id.dw=id;
    
47. pcan_msg_dst=CANMSG_BCM1_STATUS3;//复制驾驶室输入状态信息3的CAN消息
    
48. }else if(id==BCM1_DIAG1_ID)
    
49. {
    
50. CANMSG_BCM1_DIAG1[1].id.dw=id;
    
51. pcan_msg_dst=CANMSG_BCM1_DIAG1;//驾驶室诊断信息1
    
52. }else if(id==BCM1_INFO1_ID)
    
53. {
    
54. CANMSG_BCM1_INFO1[1].id.dw=id;
    
55. pcan_msg_dst=CANMSG_BCM1_INFO1;//驾驶室模块信息（输出信息）
    
56. }else if(id==DashBoard_INFO_ID)
    
57. {
    
58. CANMSG_DashBoard_INFO[1].id.dw=id;
    
59. pcan_msg_dst=CANMSG_DashBoard_INFO;//仪表模块信息（仪表模块状态信息）
    
60. }
    
61. FiltReceivdCanMsg(pcan_msg_dst,pcan_msg);
    
62. }
    
63. }
    
64. }
    
65. //中断函数                              
    
66. void interrupt 28 MSCAN_RX(void)
    
67. {
    
68.    unsigned char j;
    
69.    unsigned char arry[13];
    
70.    
    
71.    for(j = 0; j < 12; j++)
    
72.     {
    
73.        arry[j] = *(&CANRIDR0 + j);
    
74.     }   
    
75.    arry[12] = (*(&CANRIDR0 + 12) & 0x0f);
    
76.    
    
77.    CopyDataToRAM((PCAN_MSG)arry);
    
78.    CANRFLG |= 0X01;
    
79. }

复制代码

其实发现，中断并非太臃肿，原来还使用了库函数“memcpy,memcmp”,于是以为是不是库函数不合适在这里，可是去掉库函数还是不能解决问题。
最后发现一个很大的BUG,就是在函数CopyDataToRAM中，原来是这样的（未修改前）：

1. 
   
2. //复制数据到相应的CAN消息缓存区
   
3. void CopyDataToRAM(PCAN_MSG pcan_msg)
   
4. {
   
5. dword id;
   
6. PCAN_MSG pcan_msg_dst;
   
7. if (IS_BIT_SET(pcan_msg->id.b[1],4)&&IS_BIT_SET(pcan_msg->id.b[1],3)&&IS_BIT_CLR(pcan_msg->id.b[3],0))//ID满足扩展数据帧的条件
   
8. {
   
9. id=GetPureID(pcan_msg);
   
10. if(id==BCM1_STATUS1_ID)
    
11. {
    
12. CANMSG_BCM1_STATUS1[1].id.dw=id;
    
13. pcan_msg_dst=CANMSG_BCM1_STATUS1;//复制驾驶室输入状态信息1的CAN消息
    
14. }else if(id==BCM1_STATUS2_ID)
    
15. {
    
16. CANMSG_BCM1_STATUS2[1].id.dw=id;
    
17. pcan_msg_dst=CANMSG_BCM1_STATUS2;//复制驾驶室输入状态信息2的CAN消息
    
18. }else if(id==BCM1_STATUS3_ID)
    
19. {
    
20. CANMSG_BCM1_STATUS3[1].id.dw=id;
    
21. pcan_msg_dst=CANMSG_BCM1_STATUS3;//复制驾驶室输入状态信息3的CAN消息
    
22. }else if(id==BCM1_DIAG1_ID)
    
23. {
    
24. CANMSG_BCM1_DIAG1[1].id.dw=id;
    
25. pcan_msg_dst=CANMSG_BCM1_DIAG1;//驾驶室诊断信息1
    
26. }else if(id==BCM1_INFO1_ID)
    
27. {
    
28. CANMSG_BCM1_INFO1[1].id.dw=id;
    
29. pcan_msg_dst=CANMSG_BCM1_INFO1;//驾驶室模块信息（输出信息）
    
30. }else if(id==DashBoard_INFO_ID)
    
31. {
    
32. CANMSG_DashBoard_INFO[1].id.dw=id;
    
33. pcan_msg_dst=CANMSG_DashBoard_INFO;//仪表模块信息（仪表模块状态信息）
    
34. }
    
35. FiltReceivdCanMsg(pcan_msg_dst,pcan_msg);
    
36. }
    
37. }

复制代码

这样造成只要是ID满足扩展帧的条件，FiltReceivdCanMsg(pcan_msg_dst,pcan_msg);就会被执行。这样不仅造成额外的CPU占用（本来是想着如果ID不满足那几个特定ID的话就不做处理了），更重要的是如果ID不是特定的几个ID的话FiltReceivdCanMsg函数中的指针pcan_msg_dst,pcan_msg就完全跑飞了，从而造成不可预知的各种灾难性的后果。

总结：指针啊指针，逻辑啊逻辑，要谨慎！