From baca33fe20f12286a4fd3412c616c852d55a3417 Mon Sep 17 00:00:00 2001
From: sr
Date: Tue, 5 Feb 2013 18:19:31 +0100
Subject: [SHARED] Add missing error message if network message parsing fails
---
i18n/server/pvsmgr_ar_JO.ts | 4 ++--
i18n/server/pvsmgr_de_DE.ts | 4 ++--
i18n/server/pvsmgr_es_MX.ts | 4 ++--
i18n/server/pvsmgr_fr_FR.ts | 4 ++--
i18n/server/pvsmgr_pl_PL.ts | 4 ++--
src/server/mainwindow/mainwindow.cpp | 9 +++++----
src/server/net/client.cpp | 2 +-
src/shared/networkmessage.cpp | 3 +++
8 files changed, 19 insertions(+), 15 deletions(-)
diff --git a/i18n/server/pvsmgr_ar_JO.ts b/i18n/server/pvsmgr_ar_JO.ts
index 1b52528..a897589 100644
--- a/i18n/server/pvsmgr_ar_JO.ts
+++ b/i18n/server/pvsmgr_ar_JO.ts
@@ -51,12 +51,12 @@
-
+
-
+
diff --git a/i18n/server/pvsmgr_de_DE.ts b/i18n/server/pvsmgr_de_DE.ts
index ed36f2d..326374f 100644
--- a/i18n/server/pvsmgr_de_DE.ts
+++ b/i18n/server/pvsmgr_de_DE.ts
@@ -104,12 +104,12 @@
-
+
-
+
diff --git a/i18n/server/pvsmgr_es_MX.ts b/i18n/server/pvsmgr_es_MX.ts
index 2e11a13..09b67f8 100644
--- a/i18n/server/pvsmgr_es_MX.ts
+++ b/i18n/server/pvsmgr_es_MX.ts
@@ -307,12 +307,12 @@ Perform an unprojection or remove remote help to get a target.
-
+
-
+
diff --git a/i18n/server/pvsmgr_fr_FR.ts b/i18n/server/pvsmgr_fr_FR.ts
index 1b52528..a897589 100644
--- a/i18n/server/pvsmgr_fr_FR.ts
+++ b/i18n/server/pvsmgr_fr_FR.ts
@@ -51,12 +51,12 @@
-
+
-
+
diff --git a/i18n/server/pvsmgr_pl_PL.ts b/i18n/server/pvsmgr_pl_PL.ts
index 1b52528..a897589 100644
--- a/i18n/server/pvsmgr_pl_PL.ts
+++ b/i18n/server/pvsmgr_pl_PL.ts
@@ -51,12 +51,12 @@
-
+
-
+
diff --git a/src/server/mainwindow/mainwindow.cpp b/src/server/mainwindow/mainwindow.cpp
index 1d9e565..9e89baa 100644
--- a/src/server/mainwindow/mainwindow.cpp
+++ b/src/server/mainwindow/mainwindow.cpp
@@ -724,10 +724,11 @@ void MainWindow::onVncServerStateChange(Client* client)
for (QList::iterator it(_clientFrames.begin()); it != _clientFrames.end(); ++it)
{
Client *c = (**it).client();
- if (c == NULL)
- continue; // Offline
- if (c->id() == client->id() ||
- (c->desiredProjectionSource() != client->id() && !client->isProjectionSource()))
+ if (c == NULL || c->id() == client->id())
+ continue; // Offline or self
+ if (c->currentProjectionSource() == client->id())
+ continue; // Already watching this client
+ if (c->desiredProjectionSource() != client->id() && !client->isProjectionSource())
continue; // Not interested
c->sendMessage(msg);
}
diff --git a/src/server/net/client.cpp b/src/server/net/client.cpp
index 30e323a..fdabb99 100644
--- a/src/server/net/client.cpp
+++ b/src/server/net/client.cpp
@@ -143,7 +143,7 @@ void Client::handleMsg()
qDebug("Received message with empty ID field. ignored.");
return;
}
- qDebug() << "Received message " << id;
+ //qDebug() << "Received message " << id;
if (_authed == 2)
{
diff --git a/src/shared/networkmessage.cpp b/src/shared/networkmessage.cpp
index f7972f0..5e15f7b 100644
--- a/src/shared/networkmessage.cpp
+++ b/src/shared/networkmessage.cpp
@@ -113,7 +113,10 @@ bool NetworkMessage::readMessage(QAbstractSocket* socket)
const qint64 ret = socket->read(_buffer + _bufferPos, _bufferSize - _bufferPos);
//qDebug() << "Read " << ret << " bytes";
if (ret < 0)
+ {
+ qDebug("Socket read failed (TCP), return code %d", (int)ret);
return false;
+ }
_bufferPos += ret;
//qDebug() << "Buffer has now " << _bufferPos << " of " << _bufferSize << " bytes";
}
--
cgit v1.2.3-55-g7522