Throw IPC errors on invalid Mojo BlobChannel messages.

blimp/engine/mojo/blob_channel_service.cc

 // Copyright 2016 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 "blimp/engine/mojo/blob_channel_service.h"
 
 #include <string>
 #include <unordered_map>
 #include <utility>
 #include <vector>
 
 #include "base/memory/ptr_util.h"
 #include "base/task_runner_util.h"
 #include "base/threading/thread.h"
 #include "blimp/net/blob_channel/blob_channel_sender.h"
+#include "mojo/public/cpp/bindings/message.h"
 #include "mojo/public/cpp/system/buffer.h"
 
 namespace blimp {
 namespace engine {
 namespace {
 
 std::vector<BlobChannelSender::CacheStateEntry>
 GetCachedBlobIdsFromWeakPtr(base::WeakPtr<BlobChannelSender> sender) {
   if (sender) {
     return sender->GetCachedBlobIds();
@@ skipping 40 matching lines @@
   response_callback.Run(std::move(cache_state));
 }
 
 void BlobChannelService::PutBlob(const std::string& id,
                                  mojo::ScopedSharedBufferHandle data,
                                  uint32_t size) {
   DCHECK(thread_checker_.CalledOnValidThread());
 
   // Map |data| into the address space and copy out its contents.
   if (!data.is_valid()) {
-    LOG(ERROR) << "Invalid data handle received from renderer process.";
+    mojo::ReportBadMessage(
+        "Invalid data handle received from renderer process.");
     return;
   }
 
   if (size > kMaxBlobSizeBytes) {
-    LOG(ERROR) << "Blob size too big: " << size;
+    mojo::ReportBadMessage("Blob size too large.");
     return;
   }
 
   mojo::ScopedSharedBufferMapping mapping = data->Map(size);
   CHECK(mapping) << "Failed to mmap region of " << size << " bytes.";
 
   scoped_refptr<BlobData> new_blob(new BlobData);
   new_blob->data.assign(reinterpret_cast<const char*>(mapping.get()), size);
 
   blob_sender_task_runner_->PostTask(FROM_HERE,
@@ skipping 14 matching lines @@
 
 void BlobChannelService::BindRequest(
     mojo::InterfaceRequest<mojom::BlobChannel> request) {
   DCHECK(thread_checker_.CalledOnValidThread());
 
   bindings_.AddBinding(this, std::move(request));
 }
 
 }  // namespace engine
 }  // namespace blimp