由字节对齐引发的一场“血案“

最近在搞个网络通信协议，

采用socket udp传输，

运行时，居然报段错误了，

经过debug，发现居然是因为字节对齐问题导致的。

这个问题在实现通信协议，是经常会遇到的问题，

为了方便读者理解，

我把内容做了简化，分享给大家。

1、协议说明

通信协议信令格式如下：

typedef struct protocol_msg_s{UINT8 msgType;UINT8 data1;UINT8 data2;UINT16 len;char data[100];
}PRO_MSG;

根据协议格式，我造了一个数据frm，代表我收到的某个信令，

	UCHAR frm[]={0x12,0x34,0x56,0x00,0x07,0x01,0x02,0x03,0x04,0x05,0x06,0x07};

根据协议，

信令的字段与原始帧对应关系如下

于是我实现了一个简单的解析代码【该代码有问题】

int main(int argc, char **argv)
{int ret;int frm_len = 0;UINT8 frm[]={0x12,0x34,0x56,0x00,0x07,0x01,0x02,0x03,0x04,0x05,0x06,0x07};PRO_MSG *pmsg = (PRO_MSG *)frm;printf("devType:%02x data1:%02x data2:%02x len:%04x \n",pmsg->msgType,pmsg->data1,pmsg->data2,pmsg->len);
}

编译运行后，其中len的值居然是0107，而不是0007

这其实就是因为编译器采用了字节对齐导致的，

在给pmsg->len赋值时，因为需要2个字节，

这两个字节是frm[3]、frm[4]
这正好分布在两个字里，

编译器忽略了frm[3],最终将frm[4]、frm[5]合在一起赋值给了pmsg->len

为什么有字节对齐？

简单的说内存对齐能够提高 cpu 读取数据的速度，减少 cpu 访问数据的出错性（有些 cpu 必须内存对齐，否则指针访问会出错）。

原因找到了，下面就是解决了。

2、解决办法

1. 方法1 #pragma pack()

该预处理指令用来改变对齐参数。在缺省情况下，C编译器为每一个变量或数据单元按其自然对界条件分配空间。一般地，可以通过下面的方法来改变缺省的对齐参数：

使用伪指令#pragma pack (n)，C编译器将按照n字节对齐。使用伪指令#pragma pack ()，取消自定义字节对齐方式。

完整代码

#include <stdio.h>
#include <string.h>
typedef unsigned char UINT8;
typedef unsigned short UINT16;#define MAX_FRM_DATA_LEN 100
#pragma pack(1)
typedef struct protocol_msg_s{UINT8 msgType;UINT8 data1;UINT8 data2;UINT16 len;char data[MAX_FRM_DATA_LEN];
}PRO_MSG;
#pragma
int main(int argc, char **argv)
{int ret;int frm_len = 0;UINT8 frm[]={0x12,0x34,0x56,0x00,0x07,0x01,0x02,0x03,0x04,0x05,0x06,0x07};PRO_MSG *pmsg = (PRO_MSG *)frm;printf("devType:%02x data1:%02x data2:%02x len:%04x \n",pmsg->msgType,pmsg->data1,pmsg->data2,pmsg->len);
}

2. 方法2

老老实实将收到的数据帧逐字节解析，

并填充到struct protocol_msg_s

#include <stdio.h>
#include <string.h>
typedef unsigned char UINT8;
typedef unsigned short UINT16;#define MAX_FRM_DATA_LEN 100typedef struct protocol_msg_s{UINT8 msgType;UINT8 data1;UINT8 data2;UINT16 len;char data[MAX_FRM_DATA_LEN];
}PRO_MSG;int frm_parse(PRO_MSG *pmsg,UINT8 buf[],int len)
{int pos=0;pmsg->msgType = buf[pos];pos++;pmsg->data1 = buf[pos];pos++;	pmsg->data2 = buf[pos];pos++;pmsg->len = buf[pos]<<8 | buf[pos+1]<<0;pos+=2;	if(pmsg->len>MAX_FRM_DATA_LEN){printf("frm len is longer than 100\n");return -1;}memcpy(pmsg->data,&buf[pos],pmsg->len);return 0;
}
int maini(int argc, char **argv)
{int ret;int frm_len = 0;UINT8 frm[]={0x12,0x34,0x56,0x00,0x07,0x01,0x02,0x03,0x04,0x05,0x06,0x07};PRO_MSG msg;PRO_MSG *pmsg = &msg;frm_len = sizeof(frm);ret = frm_parse(pmsg,frm,frm_len);if(ret<0){printf("frm_parse fail\n");return -1;}printf("devType:%02x data1:%02x data2:%02x len:%04x \n",pmsg->msgType,pmsg->data1,pmsg->data2,pmsg->len);return 0;
}