爲什麼下面的代碼會引發SegFault？ c（Linux）

Question

這是一個可以反轉文檔數據並將其保存在同一文檔中的代碼。 但是我得到一個Segmentation Fault.Please幫助，我不知道它爲什麼會給出一個SegFault。

#include <stdio.h> 
#include <stdlib.h> 
#include <termios.h> 
#include <fcntl.h> 
#include <string.h> 
#include <unistd.h> 


int main (int argc,char* argv[]) 
{ 

    int fd,n,i,j; 

    char* buf; 

    if(argc<2) 
    printf("USAGE: %s file-to-reverse.\n",argv[0]); 

    fd=open(argv[1], O_RDWR); 
    if(fd==-1) 
    printf("ERROR: Cannot reverse %s,file does not exist.\n",argv[1]); 

    i = 0; 
    j = n-1; 

    while(i < j) 
    { 
     read(fd,buf,n); 

     char ib = buf[i]; 
     char jb = buf[j]; 

     jb = i++; 
     ib = j--; 

     write(fd,buf,n); 
    } 

    free(buf); 
    close(fd); 
} 

EDIT1 我嘗試添加：

#include <sys/stat.h> 
struct stat fs; 


    fstat(fd, &fs); 
    n= fs.st_size; 
    buf = malloc(n * sizeof (char)); 

，但現在它只是一遍又一遍，而不是 複製文檔裏面的人物倒他們。

Answer 1

關於你的第二個編輯 - 你的循環是錯誤的。

（1）將讀取的&寫出循環 - 這就是爲什麼它會再次繼續寫入&。 （2）您需要回溯到文件的開頭，否則您只需將新數據附加到文件的末尾。

（3）實際上，在寫出它們之前，必須將緩衝區中的字符反轉。

read(fd, buf, n); 

while (i < j) 
{ 

    char t = buf[i]; 

    buf[i] = buf[j]; 

    buf[j] = t; 

    i++; 
    j--; 
} 

lseek(fd, 0, SEEK_SET); 

write(fd, buf, n); 

Answer 2

您不分配，也不初始化buf。

Answer 3

您從未初始化n所以它可能是任何東西，甚至是負面的。使用fstat或其他一些方法來確定文件的大小並將其存儲在n中。

Answer 4

您的緩衝區未分配，並且n = 0，因此您將嘗試讀取0個字符。 這應該修復代碼：

buf = malloc(10 * sizeof (char)); 
n = 10; 

---

資源：

• Wikipedia - malloc()
• linux.die.net - malloc()
• linux.die.net - read()