SOCKET收包总是会收到少量错误的消息

[wb16860]

问题是这样的,在服务器端,从客户端收1种消息(消息长度,内容完全一样),先收固定长度的消息头,然后根据消息头中消息长度收消息体,再根据消息头中的消息类型进行处理.总是会出现1,2个消息类型错误的包.打印消息看了看,发现在消息内容被改写了.WIN32下以及LINUX下都测试过,都出现过这种问题,不知道是什么原因?

[hij333]

说一下，你是如何接收消息的？线程吗？
能把接收部分的关键代码贴出来吗？

[lala_benben]

应该是你的协议没对齐吧

[sunyanlu]

你的语句中有没有将受消息的变量置位清零

[sunyanlu]

或者你忽略一下信号量看看

[Maxwell]

检查你发出去的包是不是正确的，可能因为对齐或者其他什么原因，你发出去的包就是错的。

[sniperhuangwei]

检查下拆包的代码吧.

[wb16860]

回复sniperhuangwei：我把消息处理部分屏蔽掉，只进行接收还是会出现这种问题。回复hij333：是线程接收消息，其他线程处理消息，心跳等。
回复lala_benben：你所说的协议没对齐是指的什么？
回复sunyanlu：确实没清0，但是我清0后测试，问题依旧
回复maxwell：发出的包是固定的，我肯定没问题

另外，如果发包速度变小后（之前是每秒发送5000~10000的速度，改为每秒50~100），问题消失了。

[wbohome]

TCP？

[dede22]

int buf[100];
int lstart = 0;
while(1)
{
ret = recv(fd,buf+lstart,10,0);
if(ret < 0 && errno != EINTR)
{
perror("recv err:");
}
if(lstart != 10)
{
lstart += ret;
}
else
{
break;
}
}
检查一下你的 接收代码吧，或者是你发送的部分多线程同步有问题

[wb16860]

对，是TCP。
发送的包都是169个字节。

代码给大家贴上吧（win32环境）（为方便贴代码，简化了结构定义，错误处理等代码）：
#include <stdio.h>
#include <string.h>
#include <malloc.h>
#include <signal.h>
#include <io.h>
#include <tchar.h>
#include <Winsock2.h>

typedef struct MSG_HEAD_STRUCT
{
short usSize;
short usCmdId;
long slPid;
}MSG_HEAD_ST;

typedef struct MSG_BODY_STRUCT
{
short usType;//业务包类型
char msg[159];
}MSG_BODY_ST;

typedef struct MSG_STRUCT
{
MSG_HEAD_ST stHead;
MSG_BODY_ST stBody;
}MSG_ST;



SOCKADDR_IN gSynthAddr;

unsigned long ullRecvNum = 0;


SOCKET gSynthSocket;
SOCKET gSocket;
struct sockaddr_in synAddr;
int ulAddrLen = sizeof(struct sockaddr);


MSG_ST stMsg = {0};

void main()
{

int slRet;
int ulRet;
int headsize,bodysize;
int len;
char *puchead = (char *)&stMsg.stHead;
char *pucbody = (char *)&stMsg.stBody;
int i;


WSADATA gwsaSynthData;

WSAStartup(MAKEWORD(2,2),&gwsaSynthData);

gSynthSocket = socket(AF_INET, SOCK_STREAM, 0);

if(INVALID_SOCKET == gSynthSocket)
{
return;
}

/*通过配置读取*/
gSynthAddr.sin_family = AF_INET;
gSynthAddr.sin_port = htons(8586);
gSynthAddr.sin_addr.s_addr = htonl(INADDR_ANY);

slRet = bind(gSynthSocket, (SOCKADDR *)&gSynthAddr, sizeof(gSynthAddr));
if (slRet != 0)
{
return;
}

listen(gSynthSocket, 5);

ioctlsocket(gSynthSocket,FIONBIO,&ulRet); //非阻塞和阻塞都试过


while (1)
{
gSocket = accept(gSynthSocket, (SOCKADDR *)&synAddr, &ulAddrLen);
{
if (gSocket == INVALID_SOCKET)
{
continue;
}
else
{
printf("client connect success\n");
break;
}
}
}

while (1)
{
headsize = 8;
bodysize = 0;
while (headsize > 0)
{
len = recv(gSocket, puchead, headsize, 0);
if (len <= 0)
{
continue;
}
else
{
headsize -= len;
}
}


bodysize = stMsg.stHead.usSize - 8;
while (bodysize >0)
{
len = recv(gSocket, pucbody, bodysize, 0);
if (len <= 0)
{
continue;
}
else
{
bodysize -= len;
}
}

ullRecvNum++;
if (ullRecvNum % 10000 == 0)
{
printf("recv num:%d\n", ullRecvNum);
}

if (stMsg.stHead.usCmdId != 7)
{
printf("wrong msg,ullRecvNum:%d,type:%d\n", ullRecvNum, stMsg.stHead.usCmdId);
for(i = 0; i < stMsg.stHead.usSize; i++)
{
printf("0x%x,", ((char *)&stMsg)[i]);
if ((i % 19) == 0)
{
printf("\n");
}
}
printf("\n");
}

if (stMsg.stBody.usType > 7)
{
printf("gather msg type err,ullRecvNum:%d,type:%d\n", ullRecvNum, stMsg.stBody.usType);
for(i = 0; i < stMsg.stHead.usSize; i++)
{
printf("0x%x,", ((char *)&stMsg)[i]);
if ((i % 19) == 0)
{
printf("\n");
}
}
printf("\n");
}

memset((char *)&stMsg, 0, sizeof(MSG_ST));
}
}

[wbohome]

沾包？

[dede22]

headsize = 8;
bodysize = 0;
while (headsize > 0)
{
len = recv(gSocket, puchead, headsize, 0);
if (len <= 0)
{
continue;
}
else
{
headsize -= len;
}
}

红色处没有处理指针移动，数据被覆盖了
具体参看我的代码

[Jackword]

应该是对齐问题。 设置成一字节对齐 。project->settings->c/c++选项卡
Category中选Code Generation ，Struct member alignment选1Byte。

[wb16860]

搞定,问题正如dede22所说.感谢大家,结贴!!!