Why am I printing more characters that i expect?

[2kaud]

Ah - an oldy but a goodie!

The problem here is binary versus text files. When you use .seekg() to get the size of the file, this is the actual size of the file (as shown in file listings etc). But you are opening the file as text, not binary. Hence lf/cr combinations are converted to just lf etc. So the actual size of the file read is usually smaller than the size given by .seekg(). The actual number of chars read by .read() is got by using gcount(). See http://www.cplusplus.com/reference/i...stream/gcount/ Also, you don't seem to store the size of the file data buffer anywhere - so how do you know how many bytes it contains when you come to use it??

//Release the memory for the data since I coppied them into the ShaderSource structure.
		free(data);

No you haven't. You're just set a pointer to point to the memory into which the file is read - you haven't copied the memory. So you're freeing memory that is still going to used.

PS why are using malloc()/free() in c++?? At a minimum you should be using new/delete but much better to use managed memory and use unique_ptr. See http://www.cplusplus.com/reference/memory/unique_ptr/