diff --git a/src/server/game/Guilds/Guild.cpp b/src/server/game/Guilds/Guild.cpp index 9a6671027..0679e7cef 100644 --- a/src/server/game/Guilds/Guild.cpp +++ b/src/server/game/Guilds/Guild.cpp @@ -2320,9 +2320,12 @@ void Guild::DeleteMember(uint64 guid, bool isDisbanding, bool isKicked, bool can // Call script on remove before member is actually removed from guild (and database) sScriptMgr->OnGuildRemoveMember(this, player, isDisbanding, isKicked); - if (Member* member = GetMember(guid)) - delete member; - m_members.erase(lowguid); + auto memberItr = m_members.find(lowguid); + if (memberItr != m_members.end()) + { + delete memberItr->second; + m_members.erase(memberItr); + } // If player not online data in data field will be loaded from guild tabs no need to update it !! if (player) diff --git a/src/tools/map_extractor/loadlib.cpp b/src/tools/map_extractor/loadlib.cpp index 1219ef78b..5d173cb1f 100644 --- a/src/tools/map_extractor/loadlib.cpp +++ b/src/tools/map_extractor/loadlib.cpp @@ -64,7 +64,7 @@ bool FileLoader::prepareLoadedData() void FileLoader::free() { - if (data) delete[] data; + delete[] data; data = 0; data_size = 0; version = 0; diff --git a/src/tools/map_extractor/mpq_libmpq.cpp b/src/tools/map_extractor/mpq_libmpq.cpp index be8164f44..9797b3d01 100644 --- a/src/tools/map_extractor/mpq_libmpq.cpp +++ b/src/tools/map_extractor/mpq_libmpq.cpp @@ -111,7 +111,7 @@ void MPQFile::seekRelative(int offset) void MPQFile::close() { - if (buffer) delete[] buffer; + delete[] buffer; buffer = 0; eof = true; } diff --git a/src/tools/vmap4_extractor/mpq_libmpq.cpp b/src/tools/vmap4_extractor/mpq_libmpq.cpp index 70a02e5ac..5ed5e083d 100644 --- a/src/tools/vmap4_extractor/mpq_libmpq.cpp +++ b/src/tools/vmap4_extractor/mpq_libmpq.cpp @@ -111,7 +111,7 @@ void MPQFile::seekRelative(int offset) void MPQFile::close() { - if (buffer) delete[] buffer; + delete[] buffer; buffer = 0; eof = true; }