bluetooth: Require valid UUIDs when testing against Web Bluetooth Blacklist.

content/browser/bluetooth/bluetooth_blacklist_unittest.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 "content/browser/bluetooth/bluetooth_blacklist.h"
 
 #include "content/common/bluetooth/bluetooth_scan_filter.h"
 #include "device/bluetooth/bluetooth_uuid.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 40 matching lines @@
 TEST_F(BluetoothBlacklistTest, ExcludeWritesUUID) {
   BluetoothBlacklist& blacklist = BluetoothBlacklist::Get();
   BluetoothUUID exclude_writes_uuid("eeee");
   blacklist.AddOrDie(exclude_writes_uuid,
                      BluetoothBlacklist::Value::EXCLUDE_WRITES);
   EXPECT_FALSE(blacklist.IsExcluded(exclude_writes_uuid));
   EXPECT_FALSE(blacklist.IsExcludedFromReads(exclude_writes_uuid));
   EXPECT_TRUE(blacklist.IsExcludedFromWrites(exclude_writes_uuid));
 }
 
+TEST_F(BluetoothBlacklistTest, InvalidUUID) {
+  BluetoothBlacklist& blacklist = BluetoothBlacklist::Get();
+  BluetoothUUID empty_string_uuid("");
+  EXPECT_DEATH_IF_SUPPORTED(
+      blacklist.AddOrDie(empty_string_uuid, BluetoothBlacklist::Value::EXCLUDE),
+      "");
+  EXPECT_DEATH_IF_SUPPORTED(blacklist.IsExcluded(empty_string_uuid), "");
+  EXPECT_DEATH_IF_SUPPORTED(blacklist.IsExcludedFromReads(empty_string_uuid),
+                            "");
+  EXPECT_DEATH_IF_SUPPORTED(blacklist.IsExcludedFromWrites(empty_string_uuid),
+                            "");
+
+  BluetoothUUID invalid_string_uuid("Not a valid UUID string.");
+  EXPECT_DEATH_IF_SUPPORTED(
+      blacklist.AddOrDie(invalid_string_uuid,
+                         BluetoothBlacklist::Value::EXCLUDE),
+      "");
+  EXPECT_DEATH_IF_SUPPORTED(blacklist.IsExcluded(invalid_string_uuid), "");
+  EXPECT_DEATH_IF_SUPPORTED(blacklist.IsExcludedFromReads(invalid_string_uuid),
+                            "");
+  EXPECT_DEATH_IF_SUPPORTED(blacklist.IsExcludedFromWrites(invalid_string_uuid),
+                            "");
+}
+
 // Abreviated UUIDs used to create, or test against, the blacklist work
 // correctly compared to full UUIDs.
 TEST_F(BluetoothBlacklistTest, AbreviatedUUIDs) {
   BluetoothBlacklist& blacklist = BluetoothBlacklist::Get();
 
   blacklist.AddOrDie(BluetoothUUID("aaaa"), BluetoothBlacklist::Value::EXCLUDE);
   EXPECT_TRUE(blacklist.IsExcluded(
       BluetoothUUID("0000aaaa-0000-1000-8000-00805f9b34fb")));
 
   blacklist.AddOrDie(BluetoothUUID("0000bbbb-0000-1000-8000-00805f9b34fb"),
@@ skipping 175 matching lines @@
   EXPECT_FALSE(blacklist.IsExcludedFromWrites(BluetoothUUID("1801")));
   EXPECT_TRUE(blacklist.IsExcludedFromWrites(BluetoothUUID("1812")));
   EXPECT_TRUE(blacklist.IsExcludedFromWrites(BluetoothUUID("2a02")));
   EXPECT_TRUE(blacklist.IsExcludedFromWrites(BluetoothUUID("2a03")));
   EXPECT_TRUE(blacklist.IsExcludedFromWrites(BluetoothUUID("2a25")));
   EXPECT_TRUE(blacklist.IsExcludedFromWrites(BluetoothUUID("2902")));
   EXPECT_TRUE(blacklist.IsExcludedFromWrites(BluetoothUUID("2903")));
 }
 
 }  // namespace content