What is the problem with the code?

[yezi]

Hi: I am coding to output the result to the tmp file in order to send
to the client. The following is the code I write to output the system
call to a tmp file "tmp.txt". However, when running it, it generate the
message like " Segmentation fault (core dumped)"

Really confused by that . Any comments is welcomed.

tmpfile = fopen("tmp.txt" ,"a+");
if(tmpfile == NULL) {
perror("Error: can't open file.\n");
exit(1);;
}
startTm = time(NULL);
ptr = localtime(&star tTm);
//Print the time of request and Time on the server side
fprintf(tmpfile ,"The client IP is %s, The systime
is,",inet_ntoa( their_addr.sin_ addr));
fprintf(tmpfile ,asctime(ptr)); //show time problem ???
fprintf(tmpfile ,"\n The command is %s, The status is
%s\n",buf,statu s);

sep[20]="ls";
//process the buffer content
strcat(sep,"> tmp.txt");
system(sep);

[Walter Roberson]

In article <11************ *********@o13g2 000cwo.googlegr oups.com>,
yezi <ye*****@hotmai l.com> wrote:

Hi: I am coding to output the result to the tmp file in order to send
to the client. The following is the code I write to output the system
call to a tmp file "tmp.txt". However, when running it, it generate the
message like " Segmentation fault (core dumped)" fprintf(tmpfile ,"The client IP is %s, The systime
is,",inet_ntoa (their_addr.sin _addr));
fprintf(tmpfile ,asctime(ptr)); //show time problem ???

The second argument of fprintf is expected to be a format string.
If the result of asctime() happened to include a % then the fprintf
could try to convert a non-existant argument.

You could try,

fprintf(tmpfile ,"The client IP is %s, The systime is, %s",
inet_ntoa(their _addr.sin_addr) , asctime(ptr) );
As a general reminder: any time you are expecting a pointer to a
system structure or newly allocated structure to be returned, then
you should be checking for a NULL pointer return.
--
Okay, buzzwords only. Two syllables, tops. -- Laurie Anderson

[Peter Nilsson]

Walter Roberson wrote:

yezi <ye*****@hotmai l.com> wrote:

fprintf(tmpfile ,"The client IP is %s, The systime
is,",inet_ntoa (their_addr.sin _addr));
fprintf(tmpfile ,asctime(ptr)); //show time problem ???

The second argument of fprintf is expected to be a format string.
If the result of asctime() happened to include a % then the fprintf
could try to convert a non-existant argument.

You could try,

fprintf(tmpfile ,"The client IP is %s, The systime is, %s",
inet_ntoa(their _addr.sin_addr) , asctime(ptr) );
As a general reminder: any time you are expecting a pointer to a
system structure or newly allocated structure to be returned, then
you should be checking for a NULL pointer return.

Your points are valid in general, but note that asctime() cannot
return a null pointer, nor can it return a pointer to a string
containing a % character.

[inet_ntoa() is not a standard C function, so it is off-topic in clc.]

--
Peter

[Walter Roberson]

In article <11************ **********@z14g 2000cwz.googleg roups.com>,
Peter Nilsson <ai***@acay.com .au> wrote:

Your points are valid in general, but note that asctime() cannot
return a null pointer, nor can it return a pointer to a string
containing a % character.

Is it impossible for a locale to define a timezone name or day or
month name that contains a % character ?
--
When Love is gone, there's always Justice.
When Justice is gone, there's always Force.
When Force is gone, there's always Mom. -- Laurie Anderson

[Keith Thompson]

ro******@ibd.nr c-cnrc.gc.ca (Walter Roberson) writes:

In article <11************ **********@z14g 2000cwz.googleg roups.com>,
Peter Nilsson <ai***@acay.com .au> wrote:

Your points are valid in general, but note that asctime() cannot
return a null pointer, nor can it return a pointer to a string
containing a % character.

Is it impossible for a locale to define a timezone name or day or
month name that contains a % character ?

asctime() ignores the locale; it returns a string of the form

Sun Sep 16 01:03:52 1973\n\0

--
Keith Thompson (The_Other_Keit h) ks***@mib.org <http://www.ghoti.net/~kst>
San Diego Supercomputer Center <*> <http://users.sdsc.edu/~kst>
We must do something. This is something. Therefore, we must do this.

[Chris Dollin]

yezi wrote:

sep[20]="ls";
//process the buffer content
strcat(sep,"> tmp.txt");

`sep` wasn't declared, but I'm having difficulty imagining a
declaration for it that allows its 20th element to be a char*
and for `sep` itself to be a char*.

Conclusion: something is broken.

--
Chris "electric hedgehog" Dollin
"I know three kinds: hot, cool, and what-time-does-the-tune-start?"