commit a44542ca74b7da5b44ba30c205c3244805bb0600
author Serban Constantinescu <serban.constantinescu@arm.com> Thu Jan 30 15:16:45 2014 +0000
committer David Butcher <david.butcher@arm.com> Fri Jan 31 10:49:20 2014 +0000
tree 5f95a0431cb0c0cab55e21ff4bdd464d5a9684ce
parent dc832dc5513f0767c153f90a57356c3466f45dd4

ServiceManager: Add extra error handling

This patch extends the error handling. It also adds a check for a matching
binder version - kernel/userspace.

Change-Id: I43a262934b38c5711536aaa42754fed1ef04b39e
Signed-off-by: Serban Constantinescu <serban.constantinescu@arm.com>

cmds/servicemanager/service_manager.c

diff --git a/cmds/servicemanager/service_manager.c b/cmds/servicemanager/service_manager.c
index 3625e29..c8b219b 100644
--- a/cmds/servicemanager/service_manager.c
+++ b/cmds/servicemanager/service_manager.c
@@ -275,6 +275,10 @@
     void *svcmgr = BINDER_SERVICE_MANAGER;
 
     bs = binder_open(128*1024);
+    if (!bs) {
+        ALOGE("failed to open binder driver\n");
+        return -1;
+    }
 
     if (binder_become_context_manager(bs)) {
         ALOGE("cannot become context manager (%s)\n", strerror(errno));