问题描述
请有人告诉我为什么我经常得到核心转储分段? 当我选择 n=1 工作正常但问题是 n=2 和 n=3 但我不知道为什么。我想读取一个文本文件并返回一个文本、二进制或十六进制文件。
我编辑了代码并编译了它,我只收到一个警告,它指的是 %x,但我不在乎它,因为首先我必须解决 n==2。现在我没有核心转储错误。似乎没有错误,但我的第二个文件是空的。
int main(int argc,char *argv[])
{
if (argc != 3){
printf("Use this scheme: ./executabel read_file write_file -option(-b,-t,-x)\n");
exit(EXIT_FAILURE);
}
int fd1=open(argv[1],O_RDONLY);
int fd2=open(argv[2],O_WRONLY | O_CREAT);
char buffer[1024];
int rd_count = 0;
int wr_count = 0;
int n;
FILE * fpw;
fpw = fdopen (fd2,"w");
if (fd1 == -1){
printf("Failed to open the read-file.\n");
exit(EXIT_FAILURE);
}
if (fd2 == -1){
printf("Failed to open the write-file.\n");
exit(EXIT_FAILURE);
}
printf("Please choose one:\n");
printf("1. –t forces the output file to be a text one.\n");
printf("2. –b forces the output file to be binary.\n");
printf("3. –x forces the output file to be a hexadecimal representation.\n");
scanf("%d",&n);
while ((rd_count = read(fd1,(void*) buffer,1024)) > 0){
int bytes_wr = 0;
while (bytes_wr != rd_count){
if (n==1){
wr_count = write(fd2,(void*) (buffer+bytes_wr),rd_count-bytes_wr);
}else if (n==2){
wr_count = fwrite( (void*) (buffer+bytes_wr),sizeof(float),rd_count-bytes_wr,fpw);
}else{
fprintf(fpw,"%x",(void*) (buffer+bytes_wr));
}
if (wr_count < 0) {
perror("Failed to write to file");
exit(EXIT_FAILURE);
} else {
bytes_wr += wr_count;
}
}
}
if (rd_count < 0) {
perror("Failed to read from file");
exit(EXIT_FAILURE);
}
close(fd1);
close(fd2);
chmod(argv[2],777);
exit(EXIT_SUCCESS);
}
顺便提一下,这是警告
mahsa@mahsa-G53SW:~$ gcc -o ex4.o ex4.c -Wall -Wextra
ex4.c: In function ‘main’:
ex4.c:55:23: warning: format ‘%x’ expects argument of type ‘unsigned int’,but argument 3 has type ‘void *’ [-Wformat=]
55 | fprintf(fpw,(void*) (buffer+bytes_wr));
| ~^ ~~~~~~~~~~~~~~~~~~~~~~~~~
| | |
| | void *
| unsigned int
| %p
我改变这一行
fprintf(fpw,(void*) (buffer+bytes_wr));
到这里
fprintf(fpw,"%02x",buffer[bytes_wr]);
现在我什至没有警告。似乎一切正常,但当我打开第二个文件时,它是空的。
解决方法
您的代码中有多个问题:
-
main
的签名必须是int main(void)
或int main(int argc,char *argv[])
。 -
您访问缓冲区越界。
wr_count = fwrite( (void*) (buffer+bytes_wr),sizeof(float),rd_count-bytes_wr,fd2);
当您以 bytes_wr
为 0 开始并且您有一个完整的缓冲区(rd_count
容纳 1024)时,您将写入 1024 * sizeof(float)
字节,而您的缓冲区只有 1024 字节大。在这 1024 个字节之后访问任何内存位置会导致未定义的行为。
- 您的文件类型不正确: 最初由 Andreas Wenzel 指出。
int fd2=open(argv[2],O_WRONLY | O_CREAT);
...
wr_count = fwrite( (void*) (buffer+bytes_wr),fd2);
...
fprintf(fd2,"%x",(void*) (buffer+bytes_wr));
函数 fwrite
和 fprintf
需要一个 FILE*
类型的参数来标识文件。您传递类型为 int
的文件描述符。当被调用的函数将尝试取消引用预期的指针时,您会导致未定义的行为。
有两组 I/O 函数:
-
fopen
、fprintf
、fscanf
、fflush
、fwrite
、fclose
等 -
open
、write
、close
等。 一组需要FILE*
,而另一组需要int
。不要混为一谈。
你的编译器应该警告你这一点。如果那没有发生,您需要调高警告。对于 GCC,您可以使用 -Wall -Wextra
- 您没有正确打印十六进制转储:
fprintf(fd2,(void*) (buffer+bytes_wr));
您传递的地址在这里不正确。
您可能希望将单个字节传递给 fprintf
。这将是这样做的方式:
fprintf(fd2,"%02x",buffer[bytes_wr]);
还要注意包含 02
的格式字符串以指定字段的宽度。如果没有这个,你最终可能会得到 1 位数字,并且无法在输出中看到再见边界。您还应该将 buffer
更改为 unsigned char
以防止符号扩展。