commit 4ffc2eb9858fbce8558a52755b4e48a928399cce
author Nick Kralevich <nnk@google.com> Fri Aug 27 14:35:07 2010 -0700
committer Nick Kralevich <nnk@google.com> Fri Aug 27 14:37:11 2010 -0700
tree f5ed6e5f040010455e36e0cb8cf0c69efb3be3fb
parent e6ee9844ed4ae364691eb76897acc38315f6b12f

Fix bug 2950316.  Check return values.

Change-Id: I687bb5fb8195d4c1fc863e32a5e233a8b9e74196

adb.c

diff --git a/adb.c b/adb.c
index 95dc001..d9f96df 100644
--- a/adb.c
+++ b/adb.c
@@ -891,7 +891,9 @@
         struct __user_cap_header_struct header;
         struct __user_cap_data_struct cap;
 
-        prctl(PR_SET_KEEPCAPS, 1, 0, 0, 0);
+        if (prctl(PR_SET_KEEPCAPS, 1, 0, 0, 0) != 0) {
+            exit(1);
+        }
 
         /* add extra groups:
         ** AID_ADB to access the USB driver
@@ -905,11 +907,17 @@
         */
         gid_t groups[] = { AID_ADB, AID_LOG, AID_INPUT, AID_INET, AID_GRAPHICS,
                            AID_NET_BT, AID_NET_BT_ADMIN, AID_SDCARD_RW, AID_MOUNT };
-        setgroups(sizeof(groups)/sizeof(groups[0]), groups);
+        if (setgroups(sizeof(groups)/sizeof(groups[0]), groups) != 0) {
+            exit(1);
+        }
 
         /* then switch user and group to "shell" */
-        setgid(AID_SHELL);
-        setuid(AID_SHELL);
+        if (setgid(AID_SHELL) != 0) {
+            exit(1);
+        }
+        if (setuid(AID_SHELL) != 0) {
+            exit(1);
+        }
 
         /* set CAP_SYS_BOOT capability, so "adb reboot" will succeed */
         header.version = _LINUX_CAPABILITY_VERSION;