Skip to content

Conversation

@jkoritzinsky
Copy link
Member

Try changing the ICustomQueryInterface implementation to always return NotHandled instead of Failed to defer back to the ComWrappers impl instead of failing every QI (which I suspect is what is leading to the failures).

Fixes #111264

…n NotHandled instead of Failed to defer back to the ComWrappers impl.
@jkoritzinsky
Copy link
Member Author

/azp run runtime-coreclr outerloop

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@AaronRobinsonMSFT AaronRobinsonMSFT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm fine giving this a shot, but note that the failure doesn't seem to be consistent and appears limited to x86. That would seem to indicate something more than a simple return value change, no?

@jkoritzinsky
Copy link
Member Author

I think the failures were effectively from the COM apartment marshaling layer, which likely tries to QI at some point and I suspect it doesn't do well when everything specifies E_NOINTERFACE.

@jkoritzinsky jkoritzinsky merged commit c96be21 into dotnet:main Jan 30, 2025
126 of 128 checks passed
@jkoritzinsky jkoritzinsky deleted the cross-apartment-test-fix branch January 30, 2025 16:00
grendello added a commit to grendello/runtime that referenced this pull request Jan 30, 2025
* main: (31 commits)
  More native AOT Pri-1 test tree bring up (dotnet#111994)
  Fix BigInteger outerloop test (dotnet#111841)
  JIT: Run 3-opt once across all regions (dotnet#111989)
  JIT: Check for profile consistency throughout JIT backend (dotnet#111684)
  [JIT] Add legacy extended EVEX encoding and EVEX.ND/NF feature to x64 emitter backend (dotnet#108796)
  [iOS][globalization] Fix IndexOf on empty strings on iOS to return -1 (dotnet#111898)
  System.Speech: Use intellisense xml from dotnet-api-docs (dotnet#111983)
  [mono][mini] Disable inlining if we encounter class initialization failure (dotnet#111754)
  [main] Update dependencies from dotnet/roslyn (dotnet#111946)
  Update dependencies from https://github.com/dotnet/arcade build 20250129.2 (dotnet#111996)
  Try changing the ICustomQueryInterface implementation to always return NotHandled instead of Failed to defer back to the ComWrappers impl. (dotnet#111978)
  Combined dependency update (dotnet#111852)
  Replace OPTIMIZE_FOR_SIZE with feature switch (dotnet#111743)
  Fix failed assertion 'FPbased == FPbased2' (dotnet#111787)
  Add remark to `ConditionalSelect` (dotnet#111945)
  JIT: fix try region cloning when try is nested in a handler (dotnet#111975)
  Use IRootFunctions in Tensor.StdDev (dotnet#110641)
  Remove zlib dependencies from Docker containers (dotnet#111939)
  Avoid `Unsafe.As` for `Memory<T>` and `ReadOnlyMemory<T>` conversion (dotnet#111023)
  Cleanup membarrier portability (dotnet#111943)
  ...
@github-actions github-actions bot locked and limited conversation to collaborators Mar 2, 2025
@jkoritzinsky
Copy link
Member Author

/backport to release/9.0-staging

@github-actions
Copy link
Contributor

Started backporting to release/9.0-staging (link to workflow run)

@github-actions github-actions bot unlocked this conversation Dec 15, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Interop-coreclr test-bug Problem in test source code (most likely)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test failure: Interop/COM/ComWrappers/API/ComWrappersTestsBuiltInComDisabled/ComWrappersTestsBuiltInComDisabled.cmd

2 participants