quick/netradiant-custom
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

repackBSPMain(): don't check for NULL before free()

Browse Source 
reset counts (isn't required at the moment, just for potential safety)
This commit is contained in:
Garux 2020-01-20 19:33:02 +03:00
parent dfae69d446
commit d58454d718
1 changed files with 18 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
tools/quake3/q3map2/autopk3.c
View File

@ -1035,12 +1035,12 @@ int repackBSPMain( int argc, char **argv ){
//numBSPDrawVerts = 0;
Sys_Printf( "freed BSPDrawVerts\n" );
}
*/ if ( bspDrawSurfaces != 0 ) {
free( bspDrawSurfaces );
bspDrawSurfaces = NULL;
//numBSPDrawSurfaces = 0;
//Sys_Printf( "freed bspDrawSurfaces\n" );
if ( bspDrawSurfaces != 0 ) {
Sys_Printf( "freed bspDrawSurfaces\n" );
}
*/ free( bspDrawSurfaces );
bspDrawSurfaces = NULL;
numBSPDrawSurfaces = 0;
/* if ( bspLightBytes != 0 ) {
free( bspLightBytes );
bspLightBytes = NULL;
@ -1068,11 +1068,10 @@ int repackBSPMain( int argc, char **argv ){
//allocatedBSPBrushes = 0;
}
*/ if ( entities != 0 ) {
epair_t *ep2free;
for ( i = 0; i < numBSPEntities && i < numEntities; ++i ){
ep = entities[i].epairs;
while( ep != NULL){
ep2free = ep;
epair_t *ep2free = ep;
ep = ep->next;
free( ep2free );
}
@ -1091,20 +1090,20 @@ int repackBSPMain( int argc, char **argv ){
//numBSPModels = 0;
//allocatedBSPModels = 0;
}
*/ if ( bspShaders != 0 ) {
free( bspShaders );
if ( bspShaders != 0 ) {
Sys_Printf( "freed bspShaders\n" );
}
*/ free( bspShaders );
bspShaders = NULL;
//Sys_Printf( "freed bspShaders\n" );
//numBSPShaders = 0;
//allocatedBSPShaders = 0;
numBSPShaders = 0;
allocatedBSPShaders = 0;
/* if ( bspEntData != 0 ) {
Sys_Printf( "freed bspEntData\n" );
}
if ( bspEntData != 0 ) {
free( bspEntData );
*/ free( bspEntData );
bspEntData = NULL;
//Sys_Printf( "freed bspEntData\n" );
//bspEntDataSize = 0;
//allocatedBSPEntData = 0;
}
bspEntDataSize = 0;
allocatedBSPEntData = 0;
/* if ( bspNodes != 0 ) {
free( bspNodes );
bspNodes = NULL;