执行期间出现分段错误

Question

#include<stdio.h>   
int main()  
{    
    char *arg[10],*c;  
    int count=0;  
    FILE *fp,*fq;  
    printf("Name of the file:");  
    scanf("%s",arg[1]);  
    fp=fopen(arg[1],"w");  
    printf("\t\t%s",arg[1]);  
    printf("Input the text into the file\n");  
    printf("Press Ctrl+d to the stop\n");  
    while((*c=getchar())!=EOF)  
    {  
            fwrite(c,sizeof(char),1,fp);  
            count++;  
    }  
    return 0;  
}  

Answer 1

#include<stdio.h>   
int main()  
{    
   char arg[10],c;  
   int count=0;  
   FILE *fp;  
   printf("Name of the file:");  
   scanf("%s",arg);  
   fp=fopen(arg,"w");  
   printf("\t\t%s",arg);  
   printf("Input the text into the file\n");  
   printf("Press Ctrl+d to the stop\n");  
   while((c=getchar())!=EOF)  
   {  
        fwrite(&c,sizeof(char),1,fp);  
        count++;  
   }

   if(fp != NULL){
      fclose(fp);
      fp = NULL;
   }
   return 0;  
}  

Answer 2

将此行更改

char *arg[10],*c;

为

char arg[1000],c;

此行为

scanf("%s",arg[1]);  

并且

scanf("%s",arg);  

此行为

while((*c=getchar())!=EOF)

说明

while((c=getchar())!=EOF)

：

char *c; 不是字符。它是一个指向字符的指针。它一开始只是指向内存中的一个随机位，该内存通常会填充随机数据 - 无论最近写入其中的内容。

char c; 是一个字符。

同样的情况也适用于 char *arg[10]。它是一个由十个指针组成的数组。它们指向充满随机数据的随机存储器。

注意：我的更改不是最佳实践。如果有人要输入 1000 个字符或更长的文件名，您将覆盖 arg 缓冲区的末尾。根据您正在执行的操作，这可能是一个安全错误。

Answer 3

您

char *arg[10];

定义了一个由 10 个指向 char 的指针组成的数组，但没有初始化其元素。 arg[0]、arg[1]、...、arg[9] 都将具有未定义的值。

然后，您尝试将字符串输入到这些未定义的值之一中。幸运的是，你遇到了分段错误。如果您运气不好，您的程序可能会格式化您的硬盘。

Answer 4

char *arg[10] ;

arg 是 char 指针数组。在接受输入之前，您需要使用 malloc 为它们分配内存位置 -

scanf("%s",arg[1]); // arg[1] is not assigned to point to any memory location
                    // and is what causing the segmentation fault.

所以这样做 -

arg[1] = malloc( stringLengthExpectedToEnter + 1 ) ; // +1 for termination character

也应该对其余数组元素这样做（或者）简单地更改 char*arg[10] 到 char arg[10] 并确保仅输入 9 个字符。

---

我认为您混淆了指针和普通变量。

int *ptr;

ptr 是可以保存整型变量地址的变量。为ptr 变量分配内存以保存整数地址。就是这样。 ptr 处于未初始化状态，并且没有指向任何（或）可能指向垃圾的位置。取消引用未初始化指针的行为是未定义的，如果它给出分段错误，那么您就足够幸运了。

现在，您需要使用malloc为其分配一个有效的内存位置。

ptr = malloc( sizeof(int) ) ; // Allocates number of bytes required to hold an
                              // integer and returns it's address.

因此，ptr 现在指向从可以保存整数的自由存储中获取的内存位置。这些从空闲存储中获取的位置必须使用free来释放，否则就会遇到内存泄漏的经典问题。在声明时将指针初始化为 NULL 是一个很好的做法。

int *ptr = NULL ;

希望有帮助！

scanf("%d", ptr) ; // Notice that & is not required before ptr. Because ptr 
                   // content is address itself.

普通变量的故事完全不同。声明时 -

int var ;

内存被分配给 var 来保存整数。因此，您可以直接为其分配一个整数。