c/c++语言开发共享套接字数据损坏

您只需要发送缓冲区的长度（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（）。