allocating memory screws up data being read in from file

Okay where do I begin. Basically I am reading stuff in from a file. I dont' want to post all of the structures, but the following works (seemingly) perfectly:

Code:

---

for(int index = 0; index < pHeader.numMeshes; index++) {       

        fseek(filePtr, offset, SEEK_SET);
        fread(&pMesh, sizeof(Mesh), 1, filePtr);

Skin                *pSkin                = new Skin[pMesh.numSkins];
Face                *pTriangle        = new Face[pMesh.numTriangles];
TexCoord        *pTexCoord        = new TexCoord[pMesh.numVertices];
Triangle        *pVertices  = new Triangle[pMesh.numVertices * pMesh.numMeshFrames];

fread(pSkin, sizeof(Skin), pMesh.numSkins, filePtr);

fseek(filePtr, offset + pMesh.triStart, SEEK_SET);
fread(pTriangle, sizeof(Face), pMesh.numTriangles, filePtr);

fseek(filePtr, offset + pMesh.uvStart, SEEK_SET);
fread(pTexCoord, sizeof(TexCoord), pMesh.numVertices, filePtr);

fseek(filePtr, offset + pMesh.vertexStart, SEEK_SET);
fread(pVertices, sizeof(Triangle), pMesh.numVertices * pMesh.numMeshFrames, filePtr);

delete[] pSkin;
delete[] pTriangle;
delete[] pTexCoord;
delete[] pVertices;
location++;
offset += pMesh.meshSize;
}

---

Now, whenever I add this line of code into the loop the 'data' that is being read turns into garbage, i.e instead of getting sub-50 values for the specific file I am trying to read in, the range goes up into the thousands (could be reading hexadecimal memory addresses perhaps?)

m1.theObjects[location].pVertices = new CVector3[pMesh.numVertices];

m1 is a Model structure, each Model structures has a vector of type Object (NOTvector of pointers!), each Object has among other things an array of Vertices that must be allocated at run time.

Here are those data structures
//MESH
struct object {
CVector3 * pVertices; //ALL OF THE VERTICES IN THE MODEL
CVector2 * pUV; //TEXTURE COORDINATES ARE TOTALLY SWEET
char Name[256]; //I DUNNO WE MAY WANT SOME KIND OF DESCRIPTION
};

struct model {
vector<object> theObjects; //ARRAY OF MESHES
};

I dont know if I've posted enough for you to help me, but I didn't want to over do this. Anyway to sum it up allocating memory for the pVertices is what is screwing me up. When I don't allocate memory for the vertices and just read in stuff from the file and check it, it is all good.

Thanks if you can help me. I suck with allocating memory :(

EDIT:
And yes I know pHeader and pMesh arent pointers :)

your code would be much simplified if you gave that struct a constructor/destructor and did your memory allocation there. Then when the object is constructed the memory needed is allocated at the same time.

I didn't know you could give a struct a constructor/destructor, I honestly thought that was only with classes. Anyway do you think that will solve the problem? It seems my allocation of memory shouldn't be screwing up the data being read into completely different variables.

Tbh i didnt look for your error just at the design. If the design was a little better and your memory allocation was done where it should be then errors will become easier to spot. Also easier to debug each class/struct at a time with some small driver progs.
As for structs they are identical to classes in c++ with the exception that structs default access to public whereas classes default to private....

well then I guess I am just dumb because I thought structs were supposed to be methodless. I have always considered structs to just contain data members, and that if I ever needed methods I revert to classes. in my opinion it would not really matter if I made the constructor doing the allocation of memory, it is still going to be allocating the memory, but just to be fair I am not at my source and I cannot test putting the allocation in the constructor to see if it actually works.

Thanks for replying, I had my doubts as to if anyone would actually care :)

UPDATE UPDATE:
it seems using malloc instead of new DOESN'T SCREW ANYTHING UP...this is totally weird, can someone please try to explain to me WHY allocating memory in one place using the 'new' keyword was causing the data in the buffers read in from the file to be corrupted? this is stupid I hate programming.