Skip to content

Conversation

@jrabet
Copy link
Contributor

@jrabet jrabet commented Oct 8, 2020

The original XFG enlightening for _Execute_once tried to be clever by using Context to pass the final callback to the trampoline. Unfortunately, Context is used by InitOnceExecuteOnce itself and having the wrong bits set can result in an error being returned. This PR addresses this by defining a struct which contains the parameter value and callback pointer to pass those to the trampoline.

@jrabet jrabet requested a review from a team as a code owner October 8, 2020 19:11
@StephanTLavavej
Copy link
Member

@jrabet I believe you should link your account to the Microsoft organization instead of signing the CLA which is for contributors outside Microsoft.

@StephanTLavavej StephanTLavavej added the bug Something isn't working label Oct 8, 2020
Copy link
Member

@StephanTLavavej StephanTLavavej left a comment

Choose a reason for hiding this comment

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

Perfect, thank you!

Copy link
Member

@StephanTLavavej StephanTLavavej left a comment

Choose a reason for hiding this comment

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

Even more perfect!

@StephanTLavavej StephanTLavavej merged commit b6437a9 into microsoft:master Oct 8, 2020
@StephanTLavavej
Copy link
Member

Thanks for investigating and fixing this bug so quickly! 😸

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants