Fix remoting NM hosts to verify that incoming messages are dictionaries.

remoting/host/native_messaging/pipe_messaging_channel.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "remoting/host/native_messaging/pipe_messaging_channel.h"
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/callback_helpers.h"
@@ skipping 54 matching lines @@
   DCHECK(event_handler_);
 
   native_messaging_reader_.Start(
       base::Bind(&PipeMessagingChannel::ProcessMessage, weak_ptr_),
       base::Bind(&PipeMessagingChannel::Shutdown, weak_ptr_));
 }
 
 void PipeMessagingChannel::ProcessMessage(scoped_ptr<base::Value> message) {
   DCHECK(CalledOnValidThread());
 
-  if (message->GetType() != base::Value::TYPE_DICTIONARY) {
-    LOG(ERROR) << "Expected DictionaryValue";
-    Shutdown();
-    return;
-  }
-
   if (event_handler_)
     event_handler_->OnMessage(message.Pass());
 }
 
 void PipeMessagingChannel::SendMessage(
     scoped_ptr<base::Value> message) {
   DCHECK(CalledOnValidThread());
 
   bool success = message && native_messaging_writer_;
   if (success)
@@ skipping 14 matching lines @@
     // Since event_handler->OnDisconnect() will destroy the current object,
     // |event_handler_| will become a dangling pointer after OnDisconnect()
     // returns. Therefore, we set |event_handler_| to NULL beforehand.
     EventHandler* handler = event_handler_;
     event_handler_ = NULL;
     handler->OnDisconnect();
   }
 }
 
 }  // namespace remoting