binder: remove dead code in binder_get_ref_for_node
node is always non-NULL in binder_get_ref_for_node so the conditional and else clause are not needed Signed-off-by: Todd Kjos <tkjos@google.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
08dabceefe
commit
e4cffcf4bf
|
@ -670,18 +670,12 @@ static struct binder_ref *binder_get_ref_for_node(struct binder_proc *proc,
|
||||||
}
|
}
|
||||||
rb_link_node(&new_ref->rb_node_desc, parent, p);
|
rb_link_node(&new_ref->rb_node_desc, parent, p);
|
||||||
rb_insert_color(&new_ref->rb_node_desc, &proc->refs_by_desc);
|
rb_insert_color(&new_ref->rb_node_desc, &proc->refs_by_desc);
|
||||||
if (node) {
|
hlist_add_head(&new_ref->node_entry, &node->refs);
|
||||||
hlist_add_head(&new_ref->node_entry, &node->refs);
|
|
||||||
|
|
||||||
binder_debug(BINDER_DEBUG_INTERNAL_REFS,
|
binder_debug(BINDER_DEBUG_INTERNAL_REFS,
|
||||||
"%d new ref %d desc %d for node %d\n",
|
"%d new ref %d desc %d for node %d\n",
|
||||||
proc->pid, new_ref->debug_id, new_ref->desc,
|
proc->pid, new_ref->debug_id, new_ref->desc,
|
||||||
node->debug_id);
|
node->debug_id);
|
||||||
} else {
|
|
||||||
binder_debug(BINDER_DEBUG_INTERNAL_REFS,
|
|
||||||
"%d new ref %d desc %d for dead node\n",
|
|
||||||
proc->pid, new_ref->debug_id, new_ref->desc);
|
|
||||||
}
|
|
||||||
return new_ref;
|
return new_ref;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue