C 为POSIX发送消息队列时接收额外字符

我已经基本完成了一个简单的forking程序，其中一个进程通过使用mq_send和mq_receive向其父进程发送一个字符数组，并输出反向字符串。然而，由于某些原因，当我有时输入一个要发送的字符串时，程序的接收端在其中添加了字符。更重要的是，这似乎是随机的，因为再次发送相同的字符串将得到正确的输出

下面是完整的代码，以及后面的输出示例

#include  <stdio.h> /* printf */
#include  <sys/types.h> /* pid_t */
#include <unistd.h> /* get_pid */
#include <stdlib.h>     /* exit, EXIT_FAILURE */
#include <sys/wait.h>   /* wait */
#include <mqueue.h>
#include <errno.h>
#include <fcntl.h>
#include <sys/stat.h>
#include <string.h>

#define MAX_STR_LEN 500

void reverse(char s[]);
void clientFunction(const char *msgqname);
void serverFunction(const char *msgqname);
int main(void)
{
    //A
    char msgQName[] = "/queue";
    //B
    pid_t pid;
    pid = fork();
    if (pid == 0){      
        clientFunction(msgQName);
    }
    else {
        usleep(10L);
        serverFunction(msgQName);
    }
    return(0);

}
void clientFunction(const char *msgqname){
    //C

    mqd_t mqident = mq_open(msgqname, O_WRONLY|O_CREAT ,  0666, NULL);
    //D
    if (mqident == -1){
        printf("Error opening message queue. Exiting program. \n");
        exit(0);
    }
    char str[MAX_STR_LEN];  //Keep the string that has been read from input and written on the pipe
    memset(str,0,strlen(str));  
    while(1){

        printf("Enter an string:");
        gets(str);  //Reads the string from input
        printf("%i\n",strlen(str));
        unsigned len = (unsigned) strlen(str); //Finds the length of the string
        int sent = mq_send(mqident,str,len,0);
        if (sent == -1){
            printf("Error sending message. Exiting program. \n");
            exit (0);
        }
        usleep(10L);
        memset(str,0,strlen(str));
    }

}
void serverFunction(const char *msgqname){
    //F
    mqd_t mqident = mq_open (msgqname,O_RDONLY|O_CREAT , 0666, NULL);

    if (mqident == -1){
        printf("Error opening message queue. Exiting program. \n");
        printf( "Error : %s\n", strerror( errno ) );
        exit(0);
    }
    char str1[MAX_STR_LEN];
    memset(str1,0,strlen(str1));
    struct mq_attr attr;
    while(1){
        //G
        mq_getattr(mqident,&attr);
        //H
        ssize_t receive = mq_receive(mqident,str1,attr.mq_msgsize,0);
        if (receive == -1){
            printf("Error receiving message. Exiting program. \n");
            exit(0);
        }
        printf("%i, %i\n",strlen(str1), attr.mq_msgsize);
        reverse(str1);
        printf("What you wrote in reverse was:%s\n", str1); 
        memset(str1,0,strlen(str1));
    }

}
void reverse(char s[])
{
      int length = strlen(s) ;
      int c, i, j;

      for (i = 0, j = length - 1; i < j; i++, j--)
     {
     c = s[i];
     s[i] = s[j];
     s[j] = c;
      }
}

输出中的两个整数值是我用来检查长度的，第一个是发送字符串的strlen，第二个是接收字符串的strlen

我确保在初始化和printf之后立即清空字符串，因此我不知道这些神秘字符来自何处。

在void serverfunction（）和void clientFunction（）中

替换

memset（str1,0，strlen（str1））

使用

memset（str1,0，sizeof（str1））

sizeof

将给出阵列的总大小

---

现在，您可能不会收到垃圾数据。

正如您所看到的，第一次收到的字符似乎比发送的字符多4个。 原因是字符串

str1

不是以NULL结尾的，因为 初始化字符数组

str1

的

memset

。 尝试将其更改为：

memset(str1, 0, MAX_STR_LEN);

您的程序应该按预期工作

---

再想一想，如果正确发送字符串，则根本不需要

memset

。您的

get

将在

str

的末尾追加一个

'\0'

。但是

len（str）

不会计算在内。这意味着您将只发送真正的字符，而不是最后的

'\0'

因此，有两种更好的方法来解决您的问题：

另一端在收到字符串后追加最后一个

'\0'

。可以通过添加以下行来执行此操作：

str1[receive] = '\0';

就在您的

mq\u收到
之后

最后的

'\0'

也将被发送。您可以通过在

mq_send

中发送

len+1

字符来完成此操作

---

使用这两个修复程序，您可以摆脱代码中的两个

memset

调用。

@user2844080 my side over correction正常工作。你喜欢哪个操作系统和编译器using@user2844080将所有memset替换为

sizeof

。更改所有memset似乎解决了问题，非常感谢！我猜第一次没有更改的原因是因为没有更改发送函数中的strlen部分？@user2844080接受正确的答案，如果它对您有帮助的话。

str1[receive] = '\0';