[mono][debugger] Fixing AV on accessing source_file_list that is NULL (…

…dotnet#105781) * Fixing AV on accessing source_file_list that is NULL
ivdiazsa · Aug 1, 2024 · 9fe8c95 · 9fe8c95
1 parent be32947
commit 9fe8c95
Showing 1 changed file with 12 additions and 10 deletions.
diff --git a/src/mono/mono/component/debugger-agent.c b/src/mono/mono/component/debugger-agent.c
@@ -6752,16 +6752,18 @@ get_source_files_for_type (MonoClass *klass)
 
 		if (minfo) {
 			mono_debug_get_seq_points (minfo, NULL, &source_file_list, NULL, NULL, NULL);
-			for (guint j = 0; j < source_file_list->len; ++j) {
-				guint i;
-				sinfo = (MonoDebugSourceInfo *)g_ptr_array_index (source_file_list, j);
-				for (i = 0; i < files->len; ++i)
-					if (!strcmp ((const char*)g_ptr_array_index (files, i), (const char*)sinfo->source_file))
-						break;
-				if (i == files->len)
-					g_ptr_array_add (files, g_strdup (sinfo->source_file));
-			}
-			g_ptr_array_free (source_file_list, TRUE);
+			if (source_file_list != NULL) {
+				for (guint j = 0; j < source_file_list->len; ++j) {
+					guint i;
+					sinfo = (MonoDebugSourceInfo *)g_ptr_array_index (source_file_list, j);
+					for (i = 0; i < files->len; ++i)
+						if (!strcmp ((const char*)g_ptr_array_index (files, i), (const char*)sinfo->source_file))
+							break;
+					if (i == files->len)
+						g_ptr_array_add (files, g_strdup (sinfo->source_file));
+				}
+				g_ptr_array_free (source_file_list, TRUE);
+			}
 		}
 	}