pa/eqemu-server
2
0
Fork 0
mirror of https://github.com/EQEmu/Server.git synced 2026-05-16 22:58:34 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix some undefined behavior issues?

Browse Source 
MakeAnyLenString results in UB (I think?) and is aggressively optimized out with clang
GrantAlternateAdvancementAbility were missing return statements and clang had fun times with those functions too
This commit is contained in:
Michael Cook (mackal)
2016-05-06 21:31:46 -04:00
parent 3bf13c5349
commit f85add14db
2 changed files with 3 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
common/database.cpp
+1 -7
View File
@@ -860,17 +860,11 @@ void Database::GetCharName(uint32 char_id, char* name) {
}
bool Database::LoadVariables() {
char *query = nullptr;
auto results = QueryDatabase(query, LoadVariables_MQ(&query));
auto results = QueryDatabase(StringFormat("SELECT varname, value, unix_timestamp() FROM variables where unix_timestamp(ts) >= %d", varcache_lastupdate));
if (!results.Success())
{
safe_delete_array(query);
return false;
}
safe_delete_array(query);
return LoadVariables_result(std::move(results));
}