Question : atoi(getenv())

In my cgi program (written in C) for the guestbook of my homepage, several lines of codes are listed below:

   :
  printf("Content-type: text/html%c%c",10,10);  
  len=atoi(getenv("CONTENT_LENGTH"));  
   for(i=0; len && (!feof(stdin)); i++) {
                m=i;
                inputs[i].val=ReadStdin(stdin,'&',&len);
                AddToSpace(inputs[i].val);
                Convert(inputs[i].val);
                inputs[i].name=ReadData(inputs[i].val,'=');
        }                                                           :
When I try to run it, segmentation fault exists. I notice that it is the problem of the line:
  len=atoi(getenv("CONTENT_LENGTH"));  
but I don't know how to correct it. Can anyone help?

Answer : atoi(getenv())

Well that wasn't your original question but the answer is you can't set the CONTENT_LENGTH. This is set by the Web Server based on the data it recieves when the form is posted. So this means that your server isn't working correctly. (I tend to think that this is not the case).

Given that you were complaining that the seg fault was occuring on  the if(env) line and now the error message is being displayed which is on a later line then things have changed as I predicted in my previous answer and you are now dealing with a different problem.

Three things that you can try. First trun on all the warnings that you compiler will allow. Currently you either have no warnings turned on, are ignoring them or your compiler is very old.

Change the line

    FILE *fp, *fopen();

To
   FILE *fp;

As you have already included stdio.h which provides the deffinition of fopen (and your compiler should have warned you)

Change the definition of
    input inputs[10000];
to
   input inputs[1000];

As the amount of room being created on the stack may be too much depending on platform and compiler

Steve