c - 垂直冗余校验中的分段错误

Question

我正在尝试用 C 语言编写程序以进行垂直冗余检查。代码如下：

#include<stdio.h>
#include<sys/types.h>
#include<fcntl.h>
#include<unistd.h>

    int main()
    {
        int fd,i;
        char *data = "01010101010111110101010101011111";
        int count = 0,bit_count=0;
        char *parity_bit_array = NULL;
        char *data_to_send = NULL;
        char *stream_name = "Named Stream";
        do
        {
            if(*data == '1' && bit_count <= 8)
            {
                count++;
                if(bit_count == 8) 
                {
                        if( count % 2 == 0)
                        {
                            *parity_bit_array = '1';
                            count = 0;
                            bit_count = 0;
                        }
                        else
                        {
                            *parity_bit_array = '0';
                            count = 0;
                            bit_count = 0;
                        }
                }
            }
            bit_count++;
            data++;
        } while( !data);
        do
        {
            if(bit_count <= 8)
            {
                *data_to_send++ = *parity_bit_array++;
            }
            *data_to_send++ =  *data;
        } while( !data );
        printf("%s \n",data_to_send);
        mkfifo(stream_name,0666);
        fd = open(stream_name,O_WRONLY);
        write(fd,data_to_send,sizeof(data_to_send));
        close(fd);
        unlink(stream_name);
        return 0;
    }

下面显示的文件是接收方要读取其数据的发送方文件。通过使用大小数组，它可以正常工作，但我喜欢将它与指针一起使用。 此代码中的主要变量：

1. data : 要实施 VRC 的数据
2. count : 为偶校验位计数 1
3. bit_count : 计数 8 位
4. parity_bit_array：为数据中存在的每个单个字节收集奇偶校验位
5. data_to_send : data + parity_bit_array 的组合

例如： 数据： 01110000 parity_bit_array： 1 data_to_send： 01110000 1

Answer 1

您没有为 char 指针分配内存，而是尝试写入它们，这将导致未定义的行为，从而导致分段错误。

 *parity_bit_array = '1';

这段代码中有多个这样的情况。

 char *data_to_send = NULL;

data_to_send 指针永远不会分配内存，您尝试写入它

*data_to_send++ = *parity_bit_array++;

将内存分配给 char 指针，例如

 char *data_to_send = malloc(20);

在写入此数组时，如果您看到 20 个字节已被写入，只需对同一内存执行 realloc()

char *temp = realloc(data_to_send,40);
if(temp != NULL)
data_to_send = temp;

Answer 2

有多个问题：

1. } while( !data); 错了，你应该使用：} while( *data != 0);

   这可能会间接导致分段错误（如果幸运的话），使代码无法定义循环。

2. 内存未分配给 *parity_bit_array和*data_to_send。访问未分配的内存是未定义的行为，可能会导致任何事情，包括分段错误。

3. write(fd,data_to_send,sizeof(data_to_send));根据您的逻辑，应该是write(fd,data_to_send,sizeof(*data_to_send));或类似的东西。