OSDN Git Service

Do not crash on netlink manager start error
authorNingyuan Wang <nywang@google.com>
Wed, 19 Jul 2017 22:30:37 +0000 (15:30 -0700)
committerNingyuan Wang <nywang@google.com>
Wed, 19 Jul 2017 22:48:53 +0000 (22:48 +0000)
For some device(emulator) kernel netlink interface is not configured.
NetlinkManager.Start() will fail and crash wificond.
Wificond is included in base.mk so it is not feasible to remove
it from emulator image.
This patch changes the error level from FATAL to ERROR to avoid
crashing.

Bug: 63837784
Test: compile, manual test wificond works
Change-Id: I0cf3f6a1ea73e455f8362870e4c96c76b59dd41c

main.cpp

index d2a9ba4..50e8d67 100644 (file)
--- a/main.cpp
+++ b/main.cpp
@@ -139,7 +139,9 @@ int main(int argc, char** argv) {
       &OnHwBinderReadReady)) << "Failed to watch Hw Binder FD";
 
   android::wificond::NetlinkManager netlink_manager(event_dispatcher.get());
-  CHECK(netlink_manager.Start()) << "Failed to start netlink manager";
+  if (!netlink_manager.Start()) {
+    LOG(ERROR) << "Failed to start netlink manager";
+  }
   android::wificond::NetlinkUtils netlink_utils(&netlink_manager);
   android::wificond::ScanUtils scan_utils(&netlink_manager);