commit c4c5e3e820fefa13b7e0bfebf3e0cef7dd771b40
author Josh Gao <jmgao@google.com> Tue Feb 07 06:44:51 2017 +0000
committer android-build-merger <android-build-merger@google.com> Tue Feb 07 06:44:51 2017 +0000
tree 8fa9b10764ad39cba45b853cc5836cc2c368b6d8
parent 97fa79043e59a2eed1b66dc762855e62d6c37cb2
parent f6ee48e3253852c53330e30505fc6cb3d167aed0

Merge "debuggerd_handler: fix prctl return value check." am: 36b8ccb9b2
am: f18d64b2d6

Change-Id: Ic0185b883ef08d68d587346eb9a92c5703ea71c5

debuggerd/handler/debuggerd_handler.cpp

diff --git a/debuggerd/handler/debuggerd_handler.cpp b/debuggerd/handler/debuggerd_handler.cpp
index 156534e..38a7be3 100644
--- a/debuggerd/handler/debuggerd_handler.cpp
+++ b/debuggerd/handler/debuggerd_handler.cpp
@@ -218,7 +218,7 @@
     close(pipefds[1]);
 
     // Set all of the ambient capability bits we can, so that crash_dump can ptrace us.
-    for (unsigned long i = 0; prctl(PR_CAPBSET_READ, i, 0, 0, 0); ++i) {
+    for (unsigned long i = 0; prctl(PR_CAPBSET_READ, i, 0, 0, 0) != -1; ++i) {
       prctl(PR_CAP_AMBIENT, PR_CAP_AMBIENT_RAISE, i, 0, 0);
     }