IPC消息队列不适用于分叉进程

时间:2015-12-29 10:09:55

标签: c fork ipc

我试图将IPC消息队列与分叉进程一起使用,将指针传递给动态分配的字符串,但它不起作用。

这是我做的一个简单的测试。它不会打印从队列接收的字符串。但是,如果我尝试删除fork(),它就能完美运行。

#include <sys/types.h>
#include <sys/ipc.h>
#include <sys/msg.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define MSGSZ     128

typedef struct msgbuf {
    long    mtype;
    char    *mtext;
} message_buf;

int
main ()
{
    int msqid;
    char *p;
    key_t key = 129;

    message_buf sbuf, rbuf;
    p = (char *) malloc(sizeof(char) * MSGSZ);

    if ((msqid = msgget(key, IPC_CREAT|0666)) < 0) {
        perror("msgget");
        exit(1);
    }

    if (fork() == 0) {
        strcpy(p, "Did you get this?");
        sbuf.mtype = 1;
        sbuf.mtext = p;

        if (msgsnd(msqid, &sbuf, MSGSZ, IPC_NOWAIT) < 0) {
            perror("msgsnd");
            exit(1);
        }
    }
    else {
        sleep(1);

        if (msgrcv(msqid, &rbuf, MSGSZ, 0, 0) < 0) {
            perror("msgrcv");
            exit(1);
        }

        printf("Forked version: %s\n", rbuf.mtext);
        msgctl(msqid, IPC_RMID, NULL);
    }
}

1 个答案:

答案 0 :(得分:2)

问题是您正在跨进程边界发送指针。指针仅在同一过程中有效,并且在另一个过程中发送/使用时无意义。实际上,您发送指针值后跟一大堆垃圾字节,因为msgbuf.mtext实际上不是MSGSZ个字节(因此在技术上调用了未定义的行为)。

您需要做的是在消息中内联缓冲区。也就是说,将message_buf定义更改为:

typedef struct msgbuf {
    long    mtype;
    char    mtext[MSGSZ];
} message_buf;

然后直接进入mtext

strcpy(sbuf.mtext, "Did you get this?");

为清楚起见,以下是完整程序,其中描述了更改:

#include <sys/types.h>
#include <sys/ipc.h>
#include <sys/msg.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define MSGSZ     128

typedef struct msgbuf {
    long    mtype;
    char    mtext[MSGSZ];
} message_buf;

int
main (void)
{
    int msqid;
    key_t key = 129;

    message_buf sbuf, rbuf;

    if ((msqid = msgget(key, IPC_CREAT|0666)) < 0) {
        perror("msgget");
        exit(1);
    }

    if (fork() == 0) {
        strcpy(sbuf.mtext, "Did you get this?");
        sbuf.mtype = 1;

        if (msgsnd(msqid, &sbuf, MSGSZ, IPC_NOWAIT) < 0) {
            perror("msgsnd");
            exit(1);
        }
    }
    else {
        sleep(1);

        if (msgrcv(msqid, &rbuf, MSGSZ, 0, 0) < 0) {
            perror("msgrcv");
            exit(1);
        }

        printf("Forked version: %s\n", rbuf.mtext);
        msgctl(msqid, IPC_RMID, NULL);
    }
}