commit 7792d6100cce770bd350b0d1f094b498f0686177
author Alex Vakulenko <avakulenko@chromium.org> Fri Jul 24 09:17:03 2015 -0700
committer ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> Sat Jul 25 17:45:27 2015 +0000
tree da19460bc678ae1189c9e7274319c75ae8681407
parent 12affd88a0abbac46c41b8ed337d472c7ee3b589

libweave: Fix crash at buffet shutdown

The web server interface in privet portion of buffet wasn't shut down
properly and was destroyed too late after D-Bus was already uninitialized
leading to buffet crashing at shutdown.

BUG=None
TEST=`stop buffet` and observe graceful shutdown

Change-Id: I35414ead7bcb644ee3099fd552d10ea220569da8
Reviewed-on: https://chromium-review.googlesource.com/288257
Reviewed-by: Vitaly Buka <vitalybuka@chromium.org>
Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>
Tested-by: Alex Vakulenko <avakulenko@chromium.org>

libweave/src/device_manager.cc

diff --git a/libweave/src/device_manager.cc b/libweave/src/device_manager.cc
index 709f4bb..2a3e18d 100644
--- a/libweave/src/device_manager.cc
+++ b/libweave/src/device_manager.cc
@@ -32,7 +32,10 @@
 
 DeviceManager::DeviceManager() {}
 
-DeviceManager::~DeviceManager() {}
+DeviceManager::~DeviceManager() {
+  if (privet_)
+    privet_->Shutdown();
+}
 
 void DeviceManager::Start(
     const Options& options,