Memory leak in inc\MAT_Mat.gxx line 155 (OCCT 6.5.2)

Pawel
Thu, 01/12/2012 - 15:26

Forums:

Hello,

the lines

if (noofarea == 0) {
cout return;
}

do not delete the previously allocated variables: 'firstarea' and 'lastarea'. I would change the code to:

if (noofarea == 0) {
cout if(firstarea != NULL) delete [] firstarea;
if(lastarea != NULL) delete [] lastarea;
return;
}

Pawel

Paul Jimenez
Fri, 01/13/2012 - 10:45

Minor note: delete checks for NULL already, so it's unnecessary to check for it before calling it.

Pawel
Fri, 01/13/2012 - 13:00

yes, indeed.

Thank you Paul.

Pawel

Pawel
Mon, 01/30/2012 - 14:34