tree 833033d9d6e77099e5c92cbde2e7a0320f514400
parent d360e9b5f7549eecf5651ab58a5d766a095caf2e
author Christopher Wiley <wiley@chromium.org> 1424712872 -0800
committer ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> 1424737671 +0000

libchromeos: Fix race in DBusObject lifetime.

Fix a race where destroying a DBusObject with an ExportedObjectManager
caused a CHECK to fail if DBusInterface objects had not finished
claiming the corresponding DBus interfaces.  This happened because
an interface would not claim the interface until all handlers had
exported, while the parent DBusObject would unilaterally release
interfaces.  Now the DBusInterface owns the lifetime of its claimed
interface in the ExportedObjectManager.

BUG=brillo:110
TEST=Unittests.  This new unittest crashes before this change.

Change-Id: I54dcafc0c2de50983439ff6e4759280f25f05b75
Reviewed-on: https://chromium-review.googlesource.com/252092
Tested-by: Christopher Wiley <wiley@chromium.org>
Reviewed-by: Alex Vakulenko <avakulenko@chromium.org>
Commit-Queue: Christopher Wiley <wiley@chromium.org>