commit 41ec477bc70f51716b08b4a0ced18efc9e99b564
author Brian Carlstrom <bdc@google.com> Tue Oct 11 01:35:58 2011 -0700
committer Brian Carlstrom <bdc@google.com> Tue Oct 11 01:35:58 2011 -0700
tree 844133c0fde64e1baad25ef263cc9002087ab793
parent 582a7d16bb0db323d8bd730beb61578aa3765f43

Fixed DexFile.defineClass to return null, not throw Exception

Change-Id: I09e16d5530698b41f18ae33cca7745aa3d164922

src/dalvik_system_DexFile.cc

diff --git a/src/dalvik_system_DexFile.cc b/src/dalvik_system_DexFile.cc
index 96b7817..67ff27f 100644
--- a/src/dalvik_system_DexFile.cc
+++ b/src/dalvik_system_DexFile.cc
@@ -124,8 +124,6 @@
   const std::string descriptor = DotToDescriptor(class_name.c_str());
   const DexFile::ClassDef* dex_class_def = dex_file->FindClassDef(descriptor);
   if (dex_class_def == NULL) {
-    jniThrowExceptionFmt(env, "java/lang/NoClassDefFoundError", "Class %s not found",
-                         class_name.c_str());
     return NULL;
   }
 
@@ -134,6 +132,10 @@
   ClassLinker* class_linker = Runtime::Current()->GetClassLinker();
   class_linker->RegisterDexFile(*dex_file);
   Class* result = class_linker->DefineClass(descriptor, class_loader, *dex_file, *dex_class_def);
+  if (env->ExceptionCheck()) {
+    env->ExceptionClear();
+    return NULL;
+  }
   return AddLocalReference<jclass>(env, result);
 }