Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

The !DumpArray sos command emits invalid DML for array elements #13855

Merged

Conversation

tvass83
Copy link

@tvass83 tvass83 commented Sep 8, 2017

Element MethodTable is reported correctly but for individual elements, the generated DML uses the MT of the array, not the element, thus giving incorrect results. Only value types are affected.

0:007> !DumpArray /d 02b16574
Name:        WindowsFormsApplication2_SOS_bug.Test[]
MethodTable: 011c82ec
EEClass:     011c8290
Size:        412(0x19c) bytes
Array:       Rank 1, Number of elements 100, Type VALUETYPE
Element Methodtable: 011c824c
[0] 02b1657c => **here is the problem**

Element Methodtable is reported correctly but for individual elements, the generated DML uses the MT of the array, not the element, thus giving incorrect results
Only value types are affected

0:007> !DumpArray /d 02b16574
Name:        WindowsFormsApplication2_SOS_bug.Test[]
MethodTable: 011c82ec
EEClass:     011c8290
Size:        412(0x19c) bytes
Array:       Rank 1, Number of elements 100, Type VALUETYPE
Element Methodtable: 011c824c
[0] 02b1657c => here is the problem
@mikem8361
Copy link
Member

LGTM

@tvass83
Copy link
Author

tvass83 commented Sep 14, 2017

@janvorli: are contributors also allowed (encouraged?) to initiate builds via dotnet-bot ?

@janvorli
Copy link
Member

@tvass83 I think you should be able to ask dotnet-bot to rebuild a failed leg. But you should always take a look at the details of the failure before doing that and if there are errors that don't look like infrastructure issues (some Jenkins timeout etc), they should be investigated and if they are not related to the change, GitHub issue should be created before rebuilding.
This case is some infra problem, since two legs have never finished running after three days. I will restart them for you.

@janvorli
Copy link
Member

@dotnet-bot test Tizen armel Cross Debug Build please

@janvorli
Copy link
Member

@dotnet-bot test Ubuntu16.04 arm Cross Debug Build please

@janvorli
Copy link
Member

@dotnet-bot test Ubuntu x64 Checked Build and Test please

@janvorli
Copy link
Member

@dotnet-bot test OSX10.12 x64 Checked Build and Test please

@janvorli
Copy link
Member

@dotnet-bot test Ubuntu x64 Checked Build and Test please

@tvass83
Copy link
Author

tvass83 commented Oct 2, 2017

@dotnet-bot test Ubuntu armlb Cross Release Build please

@tvass83
Copy link
Author

tvass83 commented Oct 2, 2017

@dotnet-bot test Ubuntu16.04 armlb Cross Debug Build please

@janvorli janvorli merged commit fb2064e into dotnet:master Oct 2, 2017
@tvass83 tvass83 deleted the sos_printarray_fix_element_methodtable branch October 2, 2017 13:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
4 participants