Tokenizing the string problem

What I need to do is set num = 2 as it specified in server.conf. At num=atoi(strtok(NULL,"=")); I am getting segfault there or it is giving out 0. I am running linux and gcc 4.1.2.

server.conf:

Code:

---

num_of_files=2;

---

myprog.c:

Code:

---

int c, num;
char buf[100],buf1[100];
       
c=open("./server.conf",O_RDONLY);
read(c,buf,sizeof buf);
close(c);
       
buf1=strtok(buf,";");
strtok(buf1,"=");
num=atoi(strtok(NULL,"="));  //I am getting segfault here or it is giving out 0.

---

read dosn't 0-terminate the string that you read.
BTW this shouldn't compile. You're trying to assign a pointer to an arry

Code:

---

buf1=strtok(buf,";");

---

Kurt

Sorry my bad it should of been:

Code:

---

char buf[100];
char *buf1;

---

I am not sure why I get segfault or 0 here.
num=atoi(strtok(NULL,"=")); //I am getting segfault here or it is giving out 0.

Can someone help. I spent whole day trying figure this out.

Quote:

---

Originally Posted by 911help

I am not sure why I get segfault or 0 here.

---

Read this again:

Quote:

---

Originally Posted by ZuK

read dosn't 0-terminate the string that you read.

---

Rather than:

Code:

---

c=open("./server.conf",O_RDONLY);
read(c,buf,sizeof buf);
close(c);

---

Change it to this:

Code:

---

c=open("./server.conf",O_RDONLY);
read(c,buf,99);
close(c);
buf[99] ="\0";

---

Would this fix it?

Why don't you open your while using fopen with the normal text mode and use fgets to read a line from there. Rather than

Code:

---

c=open("./server.conf",O_RDONLY);

---

Perhaps doing that way your buf will be terminated by '\0' by fgets itself, reducing your overhead on you explicitly inserting a '\'0 char.

ssharish

Quote:

---

Originally Posted by 911help

Code:

---

buf[99] ="\0";

---

---

Must be:

Code:

---

buf[99] = '\0';

---

Else it won't compile. You must learn to differentiate between a string and character. "" = string, '' = character.
A string is an array of character, so when you assign to an index in the array, you must assign a character, and not a string.