Remove useless 'svcmgr_handle' (which had wrong type).

Also use the 'ptr' union member rather than 'handle'.

Signed-off-by: Serban Constantinescu <serban.constantinescu@arm.com>
Change-Id: I68e5336cd3af3bc61dbddd2b33d7e1512c0c329c
This commit is contained in:
Elliott Hughes 2015-04-03 16:51:18 -07:00
parent 1e954441d2
commit 0b41ad5d6e

View File

@ -3,6 +3,7 @@
#include <errno.h> #include <errno.h>
#include <fcntl.h> #include <fcntl.h>
#include <inttypes.h>
#include <stdio.h> #include <stdio.h>
#include <stdlib.h> #include <stdlib.h>
#include <string.h> #include <string.h>
@ -22,8 +23,6 @@
#include <cutils/log.h> #include <cutils/log.h>
#endif #endif
uint32_t svcmgr_handle;
const char *str8(const uint16_t *x, size_t x_len) const char *str8(const uint16_t *x, size_t x_len)
{ {
static char buf[128]; static char buf[128];
@ -254,10 +253,10 @@ int svcmgr_handler(struct binder_state *bs,
uint32_t strict_policy; uint32_t strict_policy;
int allow_isolated; int allow_isolated;
//ALOGI("target=%x code=%d pid=%d uid=%d\n", //ALOGI("target=%p code=%d pid=%d uid=%d\n",
// txn->target.handle, txn->code, txn->sender_pid, txn->sender_euid); // (void*) txn->target.ptr, txn->code, txn->sender_pid, txn->sender_euid);
if (txn->target.handle != svcmgr_handle) if (txn->target.ptr != BINDER_SERVICE_MANAGER)
return -1; return -1;
if (txn->code == PING_TRANSACTION) if (txn->code == PING_TRANSACTION)
@ -381,7 +380,6 @@ int main(int argc, char **argv)
cb.func_log = selinux_log_callback; cb.func_log = selinux_log_callback;
selinux_set_callback(SELINUX_CB_LOG, cb); selinux_set_callback(SELINUX_CB_LOG, cb);
svcmgr_handle = BINDER_SERVICE_MANAGER;
binder_loop(bs, svcmgr_handler); binder_loop(bs, svcmgr_handler);
return 0; return 0;