管道读数时如何计算c结构的尺寸？

Question

一个结构 s1 其中包含几个另一种结构 s2 和数量 s2 - s1_length. s2 包括一组int - arr 及其长度 -s2_length. 我想写 s1 到子进程的管道中，然后在父进程中读取。但我得到了 segmentation falut 所有的时间。我猜测是计算结构的大小有问题 s1. 但我真的找不到到底是哪里出了问题。

我的代码是这样的。

struct s1{
 int s1_length;
 struct s2 *s2;
};

struct s2{
 int s2_length;
 int *arr;
};

int getSize(int s_length, int *s2_length_arr){
    int i, sum = 0;
    for(i=0; i< s.length; i++){
        sum += sizeof(int) * (arr_length[i] + 1);
    }
    sum += sizeof(int);
    return sum;
}

void main(){
    ...
    int i;
    int *fds = malloc(sizeof(int) * 10 *2);

    for(i=0; i<10; i++)
        pipe(fds +i*2);

    struct s1 s;
    struct s2 ss;
    s.s1_length = 0;
    int s_size, sum;
    int *s2_length_arr;

    for(int i=0; i<10; i++){
       child_pid = fork();
       if(child_pid == 0){
           close(fds[i*2]);

           // do something
           // return `s1` include 10 `s2`
           s = getS(...);

           // return a int array, with the length of `arr` of every `s2`
           s2_length_arr = getS2LengthArray(...);
           sum = getSize(s.length, s2_length_arr);

           write(fds[i*2+1], &s_length, sizeof(int));
           write(fds[i*2+1], &s2_length_arr, sizeof(int)*s_length);
           write(fds[i*2+1], &s, sum);
           close(fds[i*2+1]);
           exit(0);
       } 
    for(int i=0; i<10; i++){
       close(fds[i*2+1]);
       read(fds[i*2], &s_length, sizeof(int));
       read(fds[i*2], &s2_length_arr, sizeof(int) * s_length);
       struct s1 s;
       s.s2 = malloc(sizeof(struct s2) * s_length);
       for(j=0; j<s_length; j++)
           s.s2[j].arr = malloc(sizeof(int) * s2_length_arr[j]);
       read(fds[i*2], &s, sum);
       close(fds[i*2]);
    }
}

结果，读到 s_length, s2_length_arr 和'sum'是正常的，但不能读出。s 与 segmentation fault. 有人知道这里出了什么问题吗？非常感谢。

Answer 1

一般来说，你最好把序列化代码和计算代码分开，把它放在单独的读写函数中。 所以你可能会有这样的东西。

// read/write checking for errors
void read_check(int fd, void *buffer, size_t len) {
    char *p = buffer;
    while (len > 0) {
        size_t rlen = read(fd, p, len);
        if (rlen <= 0) {
            fprintf(stderr, "Unexpected error/eof reading from %d\n", fd);
            exit(1); }
        p += rlen;
        len -= rlen }
}
void write_check(int fd, void *buffer, size_t len) {
    char *p = buffer;
    while (len > 0) {
        size_t wlen = write(fd, p, len);
        if (wlen <= 0) {
            fprintf(stderr, "Unexpected error/eof writing to %d", fd);
            exit(1); }
        p += wlen;
        len -= wlen; }
}

struct s1{
    int s1_length;
    struct s2 *s2;
};

struct s2{
    int s2_length;
    int *arr;
};

void write_s2(int fd, struct s2 *p) {
    write_check(fd, &p->s2_length, sizeof(p->s2_length));
    write_check(fd, p->arr, p->s2_length * sizeof(*p->arr));
}
void write_s1(int fd, struct s1 *p) {
    write_check(fd, &p->s1_length, sizeof(p->s1_length));
    for (int i = 0; i < p->s1_length; ++i)
        write_s2(fd, &p->s2[i]);
}
void read_s2(int fd, struct s2 *p) {
    read_check(fd, &p->s2_length, sizeof(p->s2_length));
    p->arr = malloc(p->s2_length * sizeof(*p->arr));
    if (!p->arr) {
        fprintf(stderr, "ran out of memory\n");
        exit(1); }
    read_check(fd, p->arr, p->s2_length * sizeof(*p->arr));
}
void read_s1(int fd, struct s1 *p) {
    read_check(fd, &p->s1_length, sizeof(p->s1_length));
    p->s2 = malloc(p->s1_length * sizeof(*p->s2));
    if (!p->s2) {
        fprintf(stderr, "ran out of memory\n");
        exit(1); }
    for (int i = 0; i < p->s1_length; ++i)
        read_s2(fd, &p->s2[i]);
}

然后你只需要调用 write_s1 在子代中写入整个链接数据结构，并在 read_s1 来读取它。 你可能还想写一个 free_s1 函数来释放由 read_s1

一般来说，你最终会想要一个 read_, write_ 和 free_ (也可能 create_)函数来处理你程序中的每一个结构体类型。 你还可以做一些标准化类型大小和字节顺序的事情，以允许在不同架构的机器之间交流数据结构。