commit 5516d77f61c0553f20b7332842863bc511a97074
author Steven Moreland <smoreland@google.com> Thu Jan 10 14:00:54 2019 -0800
committer Steven Moreland <smoreland@google.com> Thu Jan 10 22:02:05 2019 +0000
tree 6e848168d87a21379f6fcb4d9bf8cee728634b22
parent bcb4be70fab707a300c1a83920805e984fcb49b8

SafetyNet logging from vendor.

These weren't enabled because of the below bug causing denials.

Bug: 113083310
Bug: 121035042
Test: no denials
Change-Id: Ie32a5c99f1488fdbfcabb19aff8fa1f3ff2c7b63

cmds/servicemanager/binder.c

diff --git a/cmds/servicemanager/binder.c b/cmds/servicemanager/binder.c
index a2b4da2..cf3b172 100644
--- a/cmds/servicemanager/binder.c
+++ b/cmds/servicemanager/binder.c
@@ -154,9 +154,7 @@
 
     // fallback to original method
     if (result != 0) {
-#ifndef VENDORSERVICEMANAGER
         android_errorWriteLog(0x534e4554, "121035042");
-#endif
 
         result = ioctl(bs->fd, BINDER_SET_CONTEXT_MGR, 0);
     }

cmds/servicemanager/service_manager.c

diff --git a/cmds/servicemanager/service_manager.c b/cmds/servicemanager/service_manager.c
index ca004e9..ec3fac5 100644
--- a/cmds/servicemanager/service_manager.c
+++ b/cmds/servicemanager/service_manager.c
@@ -77,11 +77,9 @@
     ad.uid = uid;
     ad.name = name;
 
-#ifndef VENDORSERVICEMANAGER
     if (sid == NULL) {
         android_errorWriteLog(0x534e4554, "121035042");
     }
-#endif
 
     int result = selinux_check_access(sid ? sid : lookup_sid, tctx, class, perm, (void *) &ad);
     allowed = (result == 0);