diff options
| author | Elliott Hughes <enh@google.com> | 2012-12-06 16:33:38 -0800 |
|---|---|---|
| committer | Elliott Hughes <enh@google.com> | 2012-12-06 16:33:38 -0800 |
| commit | 333155b24b606fa90e7fcd54c90bd5186d24a937 (patch) | |
| tree | 5ba991fd8c67db9fd2d481a2e9a53b8d359de45d /vm/mterp | |
| parent | c2228b7feb1ed2b143666ae1a664c1cc4188ec48 (diff) | |
| download | android_dalvik-333155b24b606fa90e7fcd54c90bd5186d24a937.tar.gz android_dalvik-333155b24b606fa90e7fcd54c90bd5186d24a937.tar.bz2 android_dalvik-333155b24b606fa90e7fcd54c90bd5186d24a937.zip | |
Don't cache failures in the interface cache.
If we try to reuse a failure, we'll be holding a NULL Method* without
the current thread having the appropriate exception pending, which will
lead to a SIGSEGV.
I've added a corresponding vm-test to cts/.
Bug: http://code.google.com/p/android/issues/detail?id=29358
Change-Id: I044e438c46fce256c8eff2dac5d86778734d5614
Diffstat (limited to 'vm/mterp')
| -rw-r--r-- | vm/mterp/common/FindInterface.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/vm/mterp/common/FindInterface.h b/vm/mterp/common/FindInterface.h index 72d45ff02..23e6f0e92 100644 --- a/vm/mterp/common/FindInterface.h +++ b/vm/mterp/common/FindInterface.h @@ -31,6 +31,7 @@ INLINE Method* dvmFindInterfaceMethodInCache(ClassObject* thisClass, { #define ATOMIC_CACHE_CALC \ dvmInterpFindInterfaceMethod(thisClass, methodIdx, method, methodClassDex) +#define ATOMIC_CACHE_NULL_ALLOWED false return (Method*) ATOMIC_CACHE_LOOKUP(methodClassDex->pInterfaceCache, DEX_INTERFACE_CACHE_SIZE, thisClass, methodIdx); |