Start New Discussion within our Software Development Community

Hello,
I really don't know if this is more of a Mac of C++ question but.. I am doing a loading of a file with

short CSVload (char * filename) //called from main
{
//openning a file and checking if it exists
    FILE * datafile = NULL;
    if ((datafile=fopen (filename, "r"))==NULL)
    {
        printf ("\nError - no file found");
        return -1;
    }
//finding size of the file
    fseek (datafile,0,SEEK_END);
    int filesize = ftell (datafile);
    rewind (datafile);  
//allocating a string of a size of a file
 char * data;
    data = new char (filesize+2); //+2 just to be safe
//reading file (I am using getc but I simplified for this post) 
 fscanf (datafile,"%s", data);
//closing file
fclose (datafile);
//printing the file
 printf ("\nThe data is:\n%s",data);
//deleting data
delete [] data;
//returning success
    return 0;
}

and the program seems to be freezing every time I call this function.. I have asked somebody why this code doesn't work on OS 9.2 on G3 and the answer was that maybe I am allocating onto the system memory since OS9.2 doesn't protect it's memory like win2k or redhat 9.0 linux (my other systems). My compiler is a very old Code Warrior 7.0 Gold Edition (I usually don't use Mac's, nor this version of CW (I am only doing mac now because of my internship)).
If you can, please help,
Thank you in advance,
Ilya
:sad:

Alright, first of all... man you've really mixed C with C++ :) If you're going to use C you ought to use malloc() and free() and not new and elete. Anyway, try: char *data = (char *)malloc(sizeof(char) * (filesize + 2)); fscanf("%s",&data) (that's off the top of my head). Seriously though, use malloc and free from now on!!!!!

Sorry I really hate C++ and when it's mixed in with C is makes my heart frown

This article has been dead for over six months. Start a new discussion instead.