我正在研究一个简单的套接字客户端,它向服务器发送一个简单的字母“p”,然后从服务器读取响应。 除了一个令人困惑的问题外,它正在全力以赴。 第一次读取套接字(它发生在循环中)时,数据会出现乱码和损坏,结果如“ÿýÿû”和“μÞv”。 第一次回复后收到的所有数据都很好且有效。
我用来接收的代码是:
int n; char buffer[256]; bzero(buffer,256); strcpy(buffer, "p"); n = write(manSock,buffer,256); if (n < 0) { error("ERROR writing to management server"); } bzero(buffer,256); n = read(manSock,buffer,256); if (n < 0) { error("ERROR reading from management server"); } return buffer;
manSock
是套接字文件描述符。
有关为何发生这种情况的任何想法?
您只需要发送缓冲区的长度(strlen)。 最佳做法是在实际发送数据之前始终发送缓冲区的长度。
int len; len = strlen(buffer); n = write(manSock,&len , sizeof(int)); if (n < 0) { error("ERROR writing to len management server"); } n = write(manSock,buffer,strlen(buffer)); if (n < 0) { error("ERROR writing to management server"); } bzero(buffer,256); n = read(manSock,&len,sizeof(int)); if (n < 0) { error("ERROR reading len from management server"); } n = read(manSock,buffer,len); if (n < 0) { error("ERROR reading from management server"); }
这似乎不是与套接字有关的问题,而是与内存管理有关的问题。
您似乎返回一个指向内存的指针,该内存仅对函数本地有效。
假设你的“真实”代码看起来像这样
char * foo(void) { char buffer[256]; /* read into buffer */ return buffer; } void bar (void) { char * p = foo(); printf("%sn", p); }
然后p
指向foo()
返回后的无效内存,因为缓冲区已经在foo()
返回时被隐式释放。
解决这个问题
你现在的代码(假设它是一个函数)很糟糕,即使你接受了其他答案中给出的建议和有关write
和read
函数的注释。
原因是你返回一个本地定义的数组(实际上是第一个元素的地址), buffer
,并且这样做是未定义的行为 。 因此,如果您正在检索返回值,则buffer
不再有效,并且可以想象包含垃圾数据,即使您在函数中填充了有效数据。
C ++标记已被删除,但如果您确实使用的是C ++,则可以始终将结果复制到std::string
并返回:
std::string someFunc() { int n; char buffer[256]; //.. your code goes here //... return std::string(buffer, len); // where len is the number of characters read }
如果你正在使用C,那么让用户传递缓冲区并在函数中填写它。 不要创建本地数组并返回它们 – 这就是底线。
您完全错误地设计了API
您需要调用者提供缓冲区,您需要返回长度。 这使得您的方法签名看起来非常像recv()。 换句话说,你根本不需要这种方法。 调用者只能调用recv()。
以上就是c/c++开发分享套接字数据损坏相关内容,想了解更多C/C++开发(异常处理)及C/C++游戏开发关注计算机技术网(www.ctvol.com)!)。
本文来自网络收集,不代表计算机技术网立场,如涉及侵权请联系管理员删除。
ctvol管理联系方式QQ:251552304
本文章地址:https://www.ctvol.com/c-cdevelopment/519096.html