commit 32c15e2ef4170f3b82d90b69c52c28351ca4cf0a
author Roshan Pius <rpius@google.com> Thu Jul 07 13:46:39 2016 -0700
committer Roshan Pius <rpius@google.com> Mon Oct 24 09:23:13 2016 -0700
tree 66549101d829c937d5a66e92274237a3293e8f86
parent d6e375115b04e4ce30144a6a21ef4b1f8d3a0071

binder: Use ifname as key in BinderManager |IIface| map

Since ifname is unique, its probably a better key to lookup the
corresponding |IIface| binder object than using raw pointer.

This deprecates the usage |binder_object_key| added in |wpa_supplicant|
struct.

While there,
1. Change all comments in |BinderManager| to c++ style.
2. Add braces for all one line if statements.

BUG: 29998764
Change-Id: Ie0b571204b8363ac49c5db7666249b9a39006f31
TEST: Ran the integration tests under |wificond|.
Signed-off-by: Roshan Pius <rpius@google.com>

wpa_supplicant/binder/binder.cpp

diff --git a/wpa_supplicant/binder/binder.cpp b/wpa_supplicant/binder/binder.cpp
index eb6006c..9fee001 100644
--- a/wpa_supplicant/binder/binder.cpp
+++ b/wpa_supplicant/binder/binder.cpp
@@ -85,7 +85,7 @@
 
 int wpas_binder_register_interface(struct wpa_supplicant *wpa_s)
 {
-	if (!wpa_s->global->binder)
+	if (!wpa_s->global->binder || !wpa_s)
 		return 1;
 
 	wpa_printf(
@@ -102,7 +102,7 @@
 
 int wpas_binder_unregister_interface(struct wpa_supplicant *wpa_s)
 {
-	if (!wpa_s->global->binder)
+	if (!wpa_s->global->binder || !wpa_s)
 		return 1;
 
 	wpa_printf(