Skip to content

Validate SOS diagnostic module paths - #6037

Draft
hoyosjs wants to merge 1 commit into
dotnet:mainfrom
hoyosjs:juhoyosa/main-path-verification
Draft

hoyosjs wants to merge 1 commit into
dotnet:mainfrom
hoyosjs:juhoyosa/main-path-verification

Conversation

@hoyosjs

@hoyosjs hoyosjs commented Sep 15, 2026

Copy link
Copy Markdown
Member

Summary

  • Check DAC, DBI, cDAC, module, and PDB load paths for locality before filesystem access, signature inspection, or library loading
  • Sanitize target paths before ClrMD runtime discovery and stage remote symbol-store results locally before use
  • Preserve absolute mounted-drive paths such as Z:\ and /mnt while rejecting UNC, device, relative, and foreign-host paths
  • Add managed, native, symbol-store, and dbgshim regression coverage

Testing

  • PathUtilitiesTests: 37 passed
  • SymbolServiceTests: 4 passed
  • RuntimeLibraryProviderTests: 2 passed
  • Build.cmd -skipmanaged
  • git diff --check

Check DAC, DBI, cDAC, module, and PDB load paths for locality before filesystem access, signature inspection, or library loading.

Sanitize paths before ClrMD runtime discovery and stage remote symbol-store results locally before use. Preserve absolute mounted-drive paths such as Z:\ and /mnt so network-mounted storage remains supported.

Add regression coverage for remote, mixed-separator, drive-relative, repeated-drive-prefix, symbol staging, and dbgshim provider paths.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant