commit 4c5b265e66bebf890b6f9e53cddeb512774b1613
author Ian Rogers <irogers@google.com> Sat Dec 15 21:36:16 2012 -0800
committer Ian Rogers <irogers@google.com> Sat Dec 15 22:36:34 2012 -0800
tree 8278a6619b4dc4ec6009dcac01cf3fd4046108cb
parent 2933d538b45cae733349b81a6a1d4a6f4dc9d3ec

Test and fixes for invoke-interface on java.lang.Object methods.

Bug: 7614818

Change-Id: I704596cf3d36887e6d6d589cb3ec89e31d5b8901

src/class_linker.cc

diff --git a/src/class_linker.cc b/src/class_linker.cc
index ca8a532..6fc6a3d 100644
--- a/src/class_linker.cc
+++ b/src/class_linker.cc
@@ -3603,11 +3603,11 @@
 }
 
 AbstractMethod* ClassLinker::ResolveMethod(const DexFile& dex_file,
-                                   uint32_t method_idx,
-                                   DexCache* dex_cache,
-                                   ClassLoader* class_loader,
-                                   const AbstractMethod* referrer,
-                                   InvokeType type) {
+                                           uint32_t method_idx,
+                                           DexCache* dex_cache,
+                                           ClassLoader* class_loader,
+                                           const AbstractMethod* referrer,
+                                           InvokeType type) {
   DCHECK(dex_cache != NULL);
   // Check for hit in the dex cache.
   AbstractMethod* resolved = dex_cache->GetResolvedMethod(method_idx);
@@ -3630,6 +3630,7 @@
       break;
     case kInterface:
       resolved = klass->FindInterfaceMethod(dex_cache, method_idx);
+      DCHECK(resolved == NULL || resolved->GetDeclaringClass()->IsInterface());
       break;
     case kSuper:  // Fall-through.
     case kVirtual:
@@ -3649,6 +3650,7 @@
         break;
       case kInterface:
         resolved = klass->FindInterfaceMethod(name, signature);
+        DCHECK(resolved == NULL || resolved->GetDeclaringClass()->IsInterface());
         break;
       case kSuper:  // Fall-through.
       case kVirtual: