diff options
| author | Simon Rettberg | 2016-11-15 12:13:40 +0100 |
|---|---|---|
| committer | Simon Rettberg | 2016-11-15 12:13:40 +0100 |
| commit | eaebc23452813a8709d2bbb4d17fddb1b4f29d91 (patch) | |
| tree | 956155fc2fc5b58498a4c494cc59b7adc68bc76a /src/server/net/client.cpp | |
| parent | [server] Rewrite positioning logic of connection frames (diff) | |
| download | pvs2-eaebc23452813a8709d2bbb4d17fddb1b4f29d91.tar.gz pvs2-eaebc23452813a8709d2bbb4d17fddb1b4f29d91.tar.xz pvs2-eaebc23452813a8709d2bbb4d17fddb1b4f29d91.zip | |
Increase compiler warnings, fix a lot of those instances
- Add explicit casts
- Comment out unused params
- Remove ignored const return types
Diffstat (limited to 'src/server/net/client.cpp')
| -rw-r--r-- | src/server/net/client.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/server/net/client.cpp b/src/server/net/client.cpp index 836d49b..1b0bbdb 100644 --- a/src/server/net/client.cpp +++ b/src/server/net/client.cpp @@ -43,8 +43,9 @@ Client::Client(QSslSocket* socket) : _socket(socket) this, SLOT(onDataArrival())); // Send challenge _challenge.resize(CHALLENGE_LEN); - for (int i = 0; i < CHALLENGE_LEN; ++i) - _challenge[i] = qrand() & 0xff; + for (int i = 0; i < CHALLENGE_LEN; ++i) { + _challenge[i] = (char)(qrand() & 0xff); + } NetworkMessage msgChlng; msgChlng.setField(_ID, _CHALLENGE); msgChlng.setField(_CHALLENGE, _challenge); @@ -169,7 +170,7 @@ void Client::handleMsg() const int port = _fromClient.getFieldString("PORT").toInt(); if (port <= 0) { if (_vncPort <= 0) { - qDebug() << "Starting VNC server on client" << _name << " (" << _socket->peerAddress().toString() + _vncPort << ") failed."; + qDebug() << "Starting VNC server on client" << _name << " (" << _socket->peerAddress().toString() << ":" << QString::number(_vncPort) << ") failed."; } else { qDebug() << "Client " << _name << " stopped its VNC server"; } |