diff options
| author | Jeff Brown <jeffbrown@google.com> | 2010-03-24 18:56:57 -0700 |
|---|---|---|
| committer | Jeff Brown <jeffbrown@google.com> | 2010-03-24 19:44:50 -0700 |
| commit | 582763ae4e2910b4059dccdfd30a447e9fc974d5 (patch) | |
| tree | 8bfca3952a09ef40d5da2fe490fe36e827c0e6e8 /libs/rs/rsg_ScriptJavaClass.cpp | |
| parent | ff21e3776d588b04abee2b65e35d4daa8d112f21 (diff) | |
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
Diffstat (limited to 'libs/rs/rsg_ScriptJavaClass.cpp')
0 files changed, 0 insertions, 0 deletions