paczek/0ad
1
0
Fork 0
forked from 0ad/0ad
Code Pull Requests Activity

fix memory leak (wasn't freeing the entire chain of SMBIOS structures)

Browse Source 
This was SVN commit r9425.
This commit is contained in:
janwas 2011-05-04 13:08:09 +00:00
parent a7152270f4
commit 9719432ff9
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
source/lib/sysdep/smbios.cpp
View File

@ -126,7 +126,13 @@ static void Cleanup() // called via atexit
stringStoragePos = 0;
// free each allocated structure
#define STRUCTURE(name, id) SAFE_FREE(structures.name##_);
#define STRUCTURE(name, id)\
while(structures.name##_)\
{\
name* next = structures.name##_->next;\
SAFE_FREE(structures.name##_);\
structures.name##_ = next;\
}
STRUCTURES
#undef STRUCTURE
}
@ -417,7 +423,7 @@ static Status InitStructures()
#endif
// (instead of counting the total string size, just use the
// SMBIOS size - typically 1-2 KB - as an upper bound.)
// SMBIOS size - typically 1-2 KB - as an upper bound.)
stringStoragePos = stringStorage = (char*)calloc(table.size(), sizeof(char)); // freed in Cleanup
if(!stringStorage)
WARN_RETURN(ERR::NO_MEM);