commit 798a0e746d3f97779ccfc6bc86bfbfe854f68445
author Vitaly Buka <vitalybuka@chromium.org> Tue Jun 02 15:37:51 2015 -0700
committer ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> Wed Jun 03 21:01:32 2015 +0000
tree aca7e913e90b264ebce042bf9daa066633248382
parent 5cd7997c4f3bed58d4aad3534451f92511235f8b

privetd: Remove empty device name checks

Server allows to have empty name. Buffet and privet also has no
requirement to have it not empty.

BUG=brillo:989
CQ-DEPEND=CL:274924
TEST=`FEATURES=test emerge-gizmo privetd buffet`

Change-Id: I02e141149d6eaf36a6c92676e68ab02b8287c724
Reviewed-on: https://chromium-review.googlesource.com/274883
Reviewed-by: Vitaly Buka <vitalybuka@chromium.org>
Tested-by: Vitaly Buka <vitalybuka@chromium.org>
Commit-Queue: Vitaly Buka <vitalybuka@chromium.org>

buffet/device_registration_info.cc

diff --git a/buffet/device_registration_info.cc b/buffet/device_registration_info.cc
index 3dd5864..bd02b23 100644
--- a/buffet/device_registration_info.cc
+++ b/buffet/device_registration_info.cc
@@ -669,11 +669,7 @@
                                               const std::string& location,
                                               chromeos::ErrorPtr* error) {
   BuffetConfig::Transaction change{config_.get()};
-  if (!change.set_name(name)) {
-    chromeos::Error::AddTo(error, FROM_HERE, kErrorDomainBuffet,
-                           "invalid_parameter", "Empty device name");
-    return false;
-  }
+  change.set_name(name);
   change.set_description(description);
   change.set_location(location);
   change.Commit();