aly/servatrice
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Warning/ban notifications: null check userInfo pointer before dereferencing it

Browse source
This commit is contained in:
Fabio Bas 2016-02-24 23:47:35 +01:00
parent f31a89ebbb
commit b462f95343
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
common/server.cpp
View file

@ -268,7 +268,12 @@ QList<QString> Server::getOnlineModeratorList()
QList<QString> results; QList<QString> results;
for (int i = 0; i < clients.size(); ++i) { for (int i = 0; i < clients.size(); ++i) {
ServerInfo_User *data = clients[i]->getUserInfo(); ServerInfo_User *data = clients[i]->getUserInfo();
if (data->user_level() & ServerInfo_User::IsModerator || data->user_level() & ServerInfo_User::IsAdmin) //TODO: this line should be updated in the event there is any type of new user level created
//TODO: this line should be updated in the event there is any type of new user level created
if (data &&
(data->user_level() & ServerInfo_User::IsModerator ||
data->user_level() & ServerInfo_User::IsAdmin)
)
results << QString::fromStdString(data->name()).simplified(); results << QString::fromStdString(data->name()).simplified();
} }
return results; return results;