Hi,
Can any one tell me what's wrong with this code?
It leaks (acceding to bound checker), when it attempts to reallocate memory.
The code is not pure C, but with minor adjustments any C compiler would
compile this.
Thank you
Eitan Michaelson.
////////////////////////////////////////////////// Code Starts Here
///////////////////////////////////////////////////////////////////////
// This code attempts to create a pointer to a structure,and then allocate a
buffer for that structure, so it would hold an array of structures.
// in each loop I’m trying to reallocate the structure array so it would
grow by 1, and then I'm trying to add data into it.
// one of the structure elements is a void*, which in turn is allocated by
malloc.
#include <stdio.h>
#include <malloc.h>
#include <string.h>
#define MAX_STRUCT_SIZE (50)
struct st_Test
{
char szChar[32];
int iInt;
void *pVoid;
};
void main(void)
{
// Pointer to the structure
st_Test *pst_Test;
pst_Test = NULL; // pointer not valid yet
for(int i = 0;i <=MAX_STRUCT_SI ZE;i++)
{
if(!pst_Test) // not yet allocated
pst_Test = (st_Test*)mallo c(1 * sizeof(st_Test) );
else // Resize by 1
pst_Test = (st_Test*)reall oc(pst_Test,(i+ 1)
*sizeof(st_Test )); /// This is the Leak /////////
// Reset the buffer we got
memset(&pst_Tes t[i],'\0',sizeof(st _Test));
// Add some data to the allocated struct
pst_Test[i].iInt = 64738;
strcpy(pst_Test[i].szChar,"poke 53280,1");
// allocate buffer for the void* element ////// This is
where bound checker found the problem ///
if(!pst_Test[i].pVoid)
pst_Test[i].pVoid = (char*)malloc(3 2);
strcpy((char*)p st_Test[i].pVoid,"commodr e 64");
}
////////////////////////////////////////////////// End
///////////////////////////////////////////////////////////////////////
[ See http://www.gotw.ca/resources/clcm.htm for info about ]
[ comp.lang.c++.m oderated. First time posters: Do this! ]
Can any one tell me what's wrong with this code?
It leaks (acceding to bound checker), when it attempts to reallocate memory.
The code is not pure C, but with minor adjustments any C compiler would
compile this.
Thank you
Eitan Michaelson.
////////////////////////////////////////////////// Code Starts Here
///////////////////////////////////////////////////////////////////////
// This code attempts to create a pointer to a structure,and then allocate a
buffer for that structure, so it would hold an array of structures.
// in each loop I’m trying to reallocate the structure array so it would
grow by 1, and then I'm trying to add data into it.
// one of the structure elements is a void*, which in turn is allocated by
malloc.
#include <stdio.h>
#include <malloc.h>
#include <string.h>
#define MAX_STRUCT_SIZE (50)
struct st_Test
{
char szChar[32];
int iInt;
void *pVoid;
};
void main(void)
{
// Pointer to the structure
st_Test *pst_Test;
pst_Test = NULL; // pointer not valid yet
for(int i = 0;i <=MAX_STRUCT_SI ZE;i++)
{
if(!pst_Test) // not yet allocated
pst_Test = (st_Test*)mallo c(1 * sizeof(st_Test) );
else // Resize by 1
pst_Test = (st_Test*)reall oc(pst_Test,(i+ 1)
*sizeof(st_Test )); /// This is the Leak /////////
// Reset the buffer we got
memset(&pst_Tes t[i],'\0',sizeof(st _Test));
// Add some data to the allocated struct
pst_Test[i].iInt = 64738;
strcpy(pst_Test[i].szChar,"poke 53280,1");
// allocate buffer for the void* element ////// This is
where bound checker found the problem ///
if(!pst_Test[i].pVoid)
pst_Test[i].pVoid = (char*)malloc(3 2);
strcpy((char*)p st_Test[i].pVoid,"commodr e 64");
}
////////////////////////////////////////////////// End
///////////////////////////////////////////////////////////////////////
[ See http://www.gotw.ca/resources/clcm.htm for info about ]
[ comp.lang.c++.m oderated. First time posters: Do this! ]
Comment