corrected, unecessary walking of classes already present inside the hash table.

changed code so we avoid an extra lookup

Author: realek

mono/metadata/unity-memory-info.c

@@ -36,16 +36,18 @@ ContextRecurseClassData (CollectMetadataContext *context, MonoClass *klass)
 	* If we use g_hash_table_lookup it returns the value which we were comparing to NULL. The problem is
 	* that 0 is a valid class index and was confusing our logic.
 	*/
-	if (!g_hash_table_lookup_extended (context->allTypes, klass, &orig_key, &value))
+	if (!g_hash_table_lookup_extended (context->allTypes, klass, &orig_key, &value)) {
 		g_hash_table_insert (context->allTypes, klass, GINT_TO_POINTER (context->currentIndex++));
 
-	fieldCount = mono_class_num_fields (klass);
-	if (fieldCount > 0) {
-		while ((field = mono_class_get_fields (klass, &iter))) {
-			MonoClass *fieldKlass = mono_class_from_mono_type (field->type);
+		fieldCount = mono_class_num_fields (klass);
+		
+		if (fieldCount > 0) {
+			while ((field = mono_class_get_fields (klass, &iter))) {
+				MonoClass *fieldKlass = mono_class_from_mono_type (field->type);
 
-			if (!g_hash_table_lookup_extended (context->allTypes, fieldKlass, &orig_key, &value))
-				ContextRecurseClassData (context, fieldKlass);
+				if (fieldKlass != klass)
+					ContextRecurseClassData (context, fieldKlass);
+			}
 		}
 	}
 }