[sensors] Fix crash when nullptr instead of a pointer to sensor is sent

device/generic_sensor/sensor_provider_impl.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 "device/generic_sensor/sensor_provider_impl.h"
 
 #include <utility>
 
 #include "device/generic_sensor/platform_sensor_provider.h"
 #include "device/generic_sensor/sensor_impl.h"
 #include "mojo/public/cpp/bindings/strong_binding.h"
 
 namespace device {
 
 namespace {
 
 uint64_t GetBufferOffset(mojom::SensorType type) {
   return (static_cast<uint64_t>(mojom::SensorType::LAST) -
           static_cast<uint64_t>(type)) *
          mojom::SensorInitParams::kReadBufferSize;
 }
 
 void RunCallback(mojom::SensorInitParamsPtr init_params,
-                 SensorImpl* sensor,
+                 mojom::SensorClientRequest client,
                  const SensorProviderImpl::GetSensorCallback& callback) {
-  callback.Run(std::move(init_params), sensor->GetClient());
+  callback.Run(std::move(init_params), std::move(client));
 }
 
 void NotifySensorCreated(
     mojom::SensorInitParamsPtr init_params,
-    SensorImpl* sensor,
+    mojom::SensorClientRequest client,
     const SensorProviderImpl::GetSensorCallback& callback) {
   base::ThreadTaskRunnerHandle::Get()->PostTask(
-      FROM_HERE,
-      base::Bind(RunCallback, base::Passed(&init_params), sensor, callback));
+      FROM_HERE, base::Bind(RunCallback, base::Passed(&init_params),
+                            base::Passed(&client), callback));
 }
 
 }  // namespace
 
 // static
 void SensorProviderImpl::Create(mojom::SensorProviderRequest request) {
   PlatformSensorProvider* provider = PlatformSensorProvider::GetInstance();
   if (provider) {
     mojo::MakeStrongBinding(base::WrapUnique(new SensorProviderImpl(provider)),
                             std::move(request));
@@ skipping 43 matching lines @@
   }
 
   auto sensor_impl = base::MakeUnique<SensorImpl>(sensor);
 
   auto init_params = mojom::SensorInitParams::New();
   init_params->memory = std::move(cloned_handle);
   init_params->buffer_offset = GetBufferOffset(type);
   init_params->mode = sensor->GetReportingMode();
   init_params->default_configuration = sensor->GetDefaultConfiguration();
 
-  NotifySensorCreated(std::move(init_params), sensor_impl.get(), callback);
+  NotifySensorCreated(std::move(init_params), sensor_impl->GetClient(),
+                      callback);
 
   mojo::MakeStrongBinding(std::move(sensor_impl), std::move(sensor_request));
 }
 
 }  // namespace device