Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Set severity of rule CA1870 to warning#92135

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
MihaZupan merged 4 commits intodotnet:mainfromMihaZupan:ca1870
Sep 20, 2023
Merged

Conversation

@MihaZupan
Copy link
Member

@MihaZupanMihaZupan added this to the9.0.0 milestoneSep 15, 2023
@MihaZupanMihaZupan self-assigned thisSep 15, 2023
@ghost
Copy link

Tagging subscribers to this area: @dotnet/area-meta
See info inarea-owners.md if you want to be subscribed.

Issue Details

Followup afterdotnet/roslyn-analyzers#6898,#91727

Author:MihaZupan
Assignees:MihaZupan
Labels:

area-Meta

Milestone:9.0.0

@MihaZupan
Copy link
MemberAuthor

@MichalStrehovsky I'm seeing odd Segmentation fault failures innativeaot.CustomMain andnativeaot.GenerateUnmanagedEntryPoints -- does that look familiar / is there any chance they're related to this PR (it does touch nativeaot's corelib)?

@MichalStrehovsky
Copy link
Member

@MichalStrehovsky I'm seeing odd Segmentation fault failures innativeaot.CustomMain andnativeaot.GenerateUnmanagedEntryPoints -- does that look familiar / is there any chance they're related to this PR (it does touch nativeaot's corelib)?

If you click through the test results, you'll find a link to how to download the crash dump for this (it'srunfo get-helix-payload -j 02391e98-2ed5-4279-97a4-f74c743cbe2c -w nativeaot.SmokeTests -o c:\hell).

We're crashing on a stack overflow, the repeating part is:

 Exceptions.exe!S_P_CoreLib_System_ThrowHelper__ThrowArgumentNullException() Line 298Unknown Exceptions.exe!S_P_CoreLib_System_MemoryExtensions__IndexOfAny_7<Char>() Line 2102Unknown Exceptions.exe!S_P_CoreLib_System_MemoryExtensions__ContainsAny_7<Char>() Line 496Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_General_Helpers__EscapeTypeNameIdentifier() Line 115Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_TypeInfos_NativeFormat_NativeFormatRuntimeNamedTypeInfo__get_Name() Line 197Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_TypeInfos_RuntimeNamedTypeInfo__get_FullName() Line 96Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_TypeInfos_NativeFormat_NativeFormatRuntimeNamedTypeInfo__ToString() Line 147Unknown Exceptions.exe!S_P_CoreLib_System_Exception__AppendExceptionStackFrame() Line 141Unknown Exceptions.exe!S_P_CoreLib_System_Runtime_EH__AppendExceptionStackFrameViaClasslib() Line 270Unknown Exceptions.exe!S_P_CoreLib_System_Runtime_EH__UpdateStackTrace() Line 881Unknown Exceptions.exe!S_P_CoreLib_System_Runtime_EH__DispatchEx() Line 707Unknown Exceptions.exe!S_P_CoreLib_System_Runtime_EH__RhThrowEx() Line 641Unknown Exceptions.exe!RhpThrowEx() Line 190Unknown Exceptions.exe!S_P_CoreLib_System_ThrowHelper__ThrowArgumentNullException() Line 298Unknown

This by itself doesn't make much sense. The way we got into the repeating part is:

 Exceptions.exe!S_P_CoreLib_System_ThrowHelper__ThrowArgumentNullException() Line 298Unknown Exceptions.exe!S_P_CoreLib_System_MemoryExtensions__IndexOfAny_7<Char>() Line 2102Unknown Exceptions.exe!S_P_CoreLib_System_MemoryExtensions__ContainsAny_7<Char>() Line 496Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_General_Helpers__EscapeTypeNameIdentifier() Line 115Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_TypeInfos_NativeFormat_NativeFormatRuntimeNamedTypeInfo__get_Name() Line 197Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_TypeInfos_RuntimeNamedTypeInfo__get_FullName() Line 96Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_TypeInfos_NativeFormat_NativeFormatRuntimeNamedTypeInfo__ToString() Line 147Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_TypeInfos_RuntimeTypeInfo__EstablishDebugName() Line 739Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_TypeInfos_NativeFormat_NativeFormatRuntimeNamedTypeInfo__GetRuntimeNamedTypeInfo() Line 80Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_General_TypeUnifier__GetNamedType_0() Line 57Unknown Exceptions.exe!S_P_CoreLib_Internal_Reflection_Core_Execution_ExecutionDomain__GetNamedTypeForHandle() Line 106Unknown Exceptions.exe!S_P_Reflection_Execution_Internal_Reflection_Execution_ReflectionExecutionDomainCallbacksImplementation__GetNamedTypeForHandle() Line 40Unknown Exceptions.exe!S_P_CoreLib_Internal_Reflection_Core_NonPortable_RuntimeTypeUnifier__GetRuntimeTypeBypassCache() Line 91Unknown Exceptions.exe!S_P_CoreLib_System_Type__GetTypeFromMethodTableSlow() Line 56Unknown Exceptions.exe!S_P_CoreLib_System_Type__GetTypeFromMethodTable() Line 41Unknown Exceptions.exe!S_P_CoreLib_System_Type__GetTypeFromHandle() Line 24Unknown Exceptions.exe!UInt32__CreateChecked<Char>() Line 630Unknown Exceptions.exe!S_P_CoreLib_System_Buffers_SearchValues__TryGetSingleRange<Char>() Line 194Unknown Exceptions.exe!S_P_CoreLib_System_Buffers_SearchValues__Create_0() Line 86Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_General_Helpers___cctor() Line 27Unknown Exceptions.exe!S_P_CoreLib_System_Runtime_CompilerServices_ClassConstructorRunner__EnsureClassConstructorRun() Line 88Unknown Exceptions.exe!S_P_CoreLib_System_Runtime_CompilerServices_ClassConstructorRunner__CheckStaticClassConstructionReturnGCStaticBase() Line 35Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_General_Helpers__EscapeTypeNameIdentifier() Line 115Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_TypeInfos_NativeFormat_NativeFormatRuntimeNamedTypeInfo__get_Name() Line 197Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_TypeInfos_RuntimeNamedTypeInfo__get_FullName() Line 96Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_TypeInfos_NativeFormat_NativeFormatRuntimeNamedTypeInfo__ToString() Line 147Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_TypeInfos_RuntimeTypeInfo__EstablishDebugName() Line 739Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_TypeInfos_NativeFormat_NativeFormatRuntimeNamedTypeInfo__GetRuntimeNamedTypeInfo() Line 80Unknown Exceptions.exe!S_P_CoreLib_System_Reflection_Runtime_General_TypeUnifier__GetNamedType_0() Line 57Unknown Exceptions.exe!S_P_CoreLib_Internal_Reflection_Core_Execution_ExecutionDomain__GetNamedTypeForHandle() Line 106Unknown Exceptions.exe!S_P_Reflection_Execution_Internal_Reflection_Execution_ReflectionExecutionDomainCallbacksImplementation__GetNamedTypeForHandle() Line 40Unknown Exceptions.exe!S_P_CoreLib_Internal_Reflection_Core_NonPortable_RuntimeTypeUnifier__GetRuntimeTypeBypassCache() Line 91Unknown Exceptions.exe!S_P_CoreLib_System_Type__GetTypeFromMethodTableSlow() Line 56Unknown Exceptions.exe!S_P_CoreLib_System_Type__GetTypeFromMethodTable() Line 41Unknown Exceptions.exe!S_P_CoreLib_System_Type__GetTypeFromHandle() Line 24Unknown Exceptions.exe!S_P_CoreLib_System_Collections_Generic_Dictionary_2<System___Canon__System___Canon>___ctor_2() Line 65Unknown Exceptions.exe!S_P_CoreLib_System_Collections_Generic_Dictionary_2<System___Canon__System___Canon>___ctor_0() Line 42Unknown Exceptions.exe!S_P_CoreLib_System_AppContext__InitializeDataStore() Line 17Unknown Exceptions.exe!S_P_CoreLib_System_AppContext___cctor() Line 18Unknown

The problem is that this PR is introducing use of reflection into the implementation of reflection. That doesn't lead to anything good.

MihaZupan reacted with eyes emoji

@MihaZupan
Copy link
MemberAuthor

Thanks!
I've disabled this rule for that file in this case given it's the only problematic site.

@MihaZupanMihaZupan reopened thisSep 20, 2023
@MihaZupanMihaZupan merged commite235aef intodotnet:mainSep 20, 2023
@ghostghost locked asresolvedand limited conversation to collaboratorsOct 20, 2023
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@stephentoubstephentoubstephentoub approved these changes

@MichalStrehovskyMichalStrehovskyAwaiting requested review from MichalStrehovskyMichalStrehovsky is a code owner

Assignees

@MihaZupanMihaZupan

Labels

Projects

None yet

Milestone

9.0.0

Development

Successfully merging this pull request may close these issues.

3 participants

@MihaZupan@MichalStrehovsky@stephentoub

[8]ページ先頭

©2009-2025 Movatter.jp