需要有关文件指针分段错误的帮助

Question

我有一个“C”程序遇到了一个奇怪的问题。我在包含“feof(fp)”的行中遇到分段错误。我试图在 linux 上运行。

我什至使用 gdb 命令回溯程序..但它没有用..

检查我的示例代码..

char buf[2000],str[15],lno[5],def[15],ref[15],tmp[15],ch,ifile[20],ofile[20];

int i,j,oldi,count,c,r,d,f,t,lc=0;



FILE *fp=NULL,*fpo=NULL;

void xyzstart()
{
/*
*Some operation that is not at all concerned with the file
*
*/
}

int main()

{

printf("Enter the name of the input file\n");

gets(ifile);



fp=fopen(ifile,"r");
if(fp==NULL)

{

printf("Error");

exit(0);

}



printf("Enter the name of the output file\n");
gets(ofile);

fpo=fopen(ofile,"w");


if(fpo==NULL)

{

printf("Output file couldn't be opened\n");

exit(0);

}




while(!feof(fp))

{

fgets(buf,sizeof(buf),fp);

count++;  //Count the number of lines in a file

}



rewind(fp); //move the file pointer to the beginning of the file



while(!feof(fp)) //Error is here!! Segmentation fault (Core Dumped)!!

{

clear();  //User defined function which clears all the memory

if(count==lc)

{

nodef();  //User defined function which doesn't reads from or writes into a file

noref();  //User defined function which doesn't reads from or writes into a file

print();  //User defined function which writes the values to output file

break;

}



fgets(buf,sizeof(buf),fp); 

{

i=0;

lc++;


    while(buf[i]!=' ') //read until it encounters a space..

      {

      lno[i]=buf[i];

      i++;

      }

    lno[i]='\0';

//puts(lno);///

    }



i++;

oldi=i;

ch=buf[i];

switch(ch)

{

case 'x': xyzstart(); break;

default: printf("Nothing found");
}

}



fclose(fpo);

fclose(fp);

return 0;

}

我真的不知道该怎么办？？ 谁能帮帮我？？ 提前致谢！

这是 clear 和 xyzstart() void clear() 的代码

{

memset(buf,'\0',sizeof(buf));

memset(lno,'\0',sizeof(lno));

memset(def,'\0',sizeof(def));

memset(ref,'\0',sizeof(ref));

i=oldi=0;

memset(str,'\0',sizeof(str));

}

void xyzstart()

{

r=d=c=0;

for(;;c++,i++)

       {

           if(buf[i]==' ')

           break;

           if(buf[i]=='(') break;

           if(buf[i]==';')break;

           if(buf[i]=='\n') break;

           if(buf[i]=='=') break;

           if(buf[i]=='+' || buf[i]=='-') break;

           str[c]=buf[i];

        }

        str[c]='\0';

if(buf[i]=='=')

           assignment();

else if(buf[i]=='+' || buf[i]=='-') //Increments or decrements

            incdec();

          else if(buf[i]=='(')

                udefined();

}

Answer 1

我的猜测是，在while中调用的函数之一正在破坏文件指针（可能会关闭它）。

主要嫌疑人： clear和xyzstart 。

Answer 2

一些注意事项：

1. 不要使用feof(fp)作为你的 while 循环条件； function 只有在您尝试读取文件末尾之后才会返回 true，因此您会经常循环一次。 检查输入操作的结果（ fgets将在失败时返回 NULL ）然后测试 EOF，如下所示：

    while(fgets(buf, sizeof buf, fp);= NULL) count++; if (feof(fp)) printf("At end of file\n");

2. NEVER NEVER NEVER NEVER NEVER NEVER使用gets ：它会在你的程序中引入一个故障点（在这种情况下可能有）。 从 C99 开始，它已被弃用，预计将从该语言的下一个版本中消失（是的，这个库调用造成的混乱比破坏 30 多年遗留代码的前景更可怕）。 请改用fgets或其他替代方法。

从您发布的代码中，我没有看到明显的问题； 如果前面的语句是成功的rewind ，我不知道为什么feof会核心转储。 我唯一能想到的是文件指针正在某处被覆盖（可能是由于gets调用中的缓冲区溢出）。

Answer 3

1. 实际上“明确”在做什么？

2. 这个循环看起来很危险：

   while(buf[i].=' ') //读取直到遇到空格..

如果 buf 不包含空格怎么办？