Use only DIPs size in ClientResolution.

remoting/protocol/host_control_dispatcher.cc

Index: remoting/protocol/host_control_dispatcher.cc
diff --git a/remoting/protocol/host_control_dispatcher.cc b/remoting/protocol/host_control_dispatcher.cc
index 7f3ea272e1affdf4c0aa8281ce3834115fe84815..3e5260cfab7eb736b3d214e37169a28128835b85 100644
--- a/remoting/protocol/host_control_dispatcher.cc
+++ b/remoting/protocol/host_control_dispatcher.cc
@@ -75,7 +75,13 @@ void HostControlDispatcher::OnIncomingMessage(
   if (message->has_clipboard_event()) {
     clipboard_stub_->InjectClipboardEvent(message->clipboard_event());
   } else if (message->has_client_resolution()) {
-    host_stub_->NotifyClientResolution(message->client_resolution());
+    const ClientResolution& resolution = message->client_resolution();
+    if (!resolution.has_dips_width() || !resolution.has_dips_height() ||
+        resolution.dips_width() <= 0 || resolution.dips_height() <= 0) {
+      LOG(ERROR) << "Received invalid ClientResolution message.";
+      return;
+    }
+    host_stub_->NotifyClientResolution(resolution);

[Jamie, 2016/05/31 20:40:47]

It seems odd that this is the only message we validate at this level. If you
think that this is the right place to validate, then this is fine as-is, but I
think we should add validation for the other messages here as well (not in this
CL).

[Sergey Ulanov, 2016/06/01 10:08:51]

Agree. Added a TODO.

   } else if (message->has_video_control()) {
     host_stub_->ControlVideo(message->video_control());
   } else if (message->has_audio_control()) {