tree 8bfca3952a09ef40d5da2fe490fe36e827c0e6e8
parent ff21e3776d588b04abee2b65e35d4daa8d112f21
author Jeff Brown <jeffbrown@google.com> 1269482217 -0700
committer Jeff Brown <jeffbrown@google.com> 1269485090 -0700

Ensure Binder finalizer handles partially initialized instances.

If the Binder is allocated but its constructor does not run for some
reason, then Binder.init() will not be called.  Since the object was
allocated, it is still eligible for finalization.  Eventually when
the finalizer runs and calls Binder.destroy(), it will have a NULL
binder holder pointer.  Previously this would cause Binder.destroy()
to attempt to decrement a reference count on a NULL pointer.

Now we check and ignore the binder if it does not have a valid
holder pointer.

Bug: b/2533956
Change-Id: Ifc2729b2f2abe8bceea5a0645ae0a4c1575b7846