Thread: How to remove Segmentation Fault

Well, you don't bother checking the return value of fopen. Could that be it?

Oh, and you open "comm.txt" twice! Once in main and once in mac.

And what's up with all the pointless globals?

Also, if someone wanted to test your code, they would need the data files.

Segmentation fault is usually doing something with memory that is pointing to NULL

Check to see if your files open before using them.

fp1 is opened twice in main and mac, you shouldn't do that.

mac() closes fp1 and then you try to use it in main - That could be it

A solution to this would be if you declare local file pointers, not global - Or at least a local file pointer for your operations in mac()

Okay thanks
I declared the file pointers local, yet the compiler shows segmentation error.
The 2 files are simply any arbitrary files with comm.txt being a c program file.
Please help.

The following is the edited code:

Code:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>


int i;
char str[10];
char str1[10],str2[10];


void mac (void) {
	FILE *fp1;


	char ch;


	fp1=fopen("comm.txt","r");
if (fp1==NULL)
{
printf("Cant open file!");
exit(0);
}
	do{
		ch=fgetc(fp1);
		if(ch!='\n')
		{
			while(ch!='\n')
			{
				ch=fgetc(fp1);
			}
		}
		ch=fgetc(fp1);
		if (ch=='#')
		{
			ch=fgetc(fp1);
			if(ch==' ')
				{
				i=0;
				ch=fgetc(fp1);
				while (ch!=' ')
					{
					str1[i++]=ch;
					ch=fgetc(fp1);
					}
				str1[i]='\0';
				}
			//puts(str1);
			if(!(strcmp(str1,"define")))
			{


				i=0;
				ch=fgetc(fp1);
				while(ch!=' ' && ch!='\n')
				{
					str2[i++]=ch;
					ch=fgetc(fp1);
				}


				str2[i]='\0';


			}


		}
	} while(ch!= EOF);


	fclose(fp1);






}


int main(void)
{
	char ch;
	FILE *fp1, *fp2;
if (fp1==NULL)
{
printf("Cant open file!");
exit(0);
}


	int flag=1;
	fp1=fopen("comm.txt","r");
	fp2=fopen("new.txt","w");
	mac();
	do{
		if(flag==0)
			{
				ch=fgetc(fp1);
			}
			
		if (ch!='#')
		{
			while(ch!='\n' && ch!=';')


			{
				i=0;
				while(ch!=' ' && ch!='\n' && ch!=';')
				{
				str[i++]=ch;
				ch=fgetc(fp1);
				}


			str[i]='\0';
			//ch=fgetc(fp1);
			if (!(strcmp(str,str1)))
				fputs(str2,fp2);
			else
				fputs(str,fp2);
			fputc(ch,fp1);
			ch=fgetc(fp1);
			}
			fputc(ch,fp1);
			if (ch=='\n')
				flag=1;
			else
				flag =0;
		}
		else
		{
		while(ch!='\n')
				{
				fputc(ch,fp2);
				ch=fgetc(fp1);


				}
		fputc(ch,fp2);
		flag=1;
		}


	}while(ch!=EOF);
	fclose(fp1);
	fclose(fp2);




	return EXIT_SUCCESS;
}

#1.

Code:

void mac (void) {
    FILE *fp1;
 
 
    char ch;

    ...

    ch=fgetc(fp1);

    ...

    } while(ch!= EOF);

The fgetc function returns an int, not a char. This is very important when comparing against EOF. The var ch should be declared as an int. You do this both in main and in mac.

---

#2
At the top of the do-while loop in function mac, all this:

Code:

ch=fgetc(fp1);
if(ch!='\n')
{
    while(ch!='\n')
    {
        ch=fgetc(fp1);
    }
}

Can be safely replaced simply with this:

Code:

while((ch=fgetc(fp1)) != '\n');

In the start of your main:

Code:

int main(void)
{
    char ch;
    FILE *fp1, *fp2;
    if (fp1==NULL)
    {
        printf("Cant open file!");
        exit(0);
    }




    int flag=1;
    fp1=fopen("comm.txt","r");
    fp2=fopen("new.txt","w");


    ...

I think that you've typed "if (fp1==NULL)" in the wrong spot and you don't check your new.txt has opened. Also, you have 2 versions of comm.txt open at the same time: Don't open comm.txt until after the mac() subroutine has been called.


It should be something more like this

Code:

int main(void)
{
    //char ch; Replaced by integer type as hk_mp5kpdw described...
    int in;


    FILE *fp1, *fp2;
    int flag=1;


    mac();


    fp1=fopen("comm.txt","r");
    fp2=fopen("new.txt","w");


    if ((fp1==NULL) || (fp2 == NULL))
    {
        printf("Cant open file!");
        exit(0);
    }




    ...

Turn all of your global variables into local variables, fix your indentation, and apply other's advice,
then I'll take a look at it.