Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix NullReferenceException in Invariant.get_IsDialogOverrideEnabled #6785

Conversation

VladimirKhil
Copy link
Contributor

@VladimirKhil VladimirKhil commented Jul 12, 2022

Fixes #6772

Description

Fix NullReferenceException in Invariant.get_IsDialogOverrideEnabled when .NET Framework Registry key (HKLM\Software\Microsoft.NETFramework) is not present on the machine

Customer Impact

Regression

Testing

Risk

Microsoft Reviewers: Open in CodeFlow

…hen .NET Framework Registry key (HKLM\Software\Microsoft\.NETFramework) is not present on the machine

dotnet#6772
@VladimirKhil VladimirKhil requested a review from a team as a code owner July 12, 2022 19:30
@ghost ghost assigned VladimirKhil Jul 12, 2022
@ghost ghost added the PR metadata: Label to tag PRs, to facilitate with triage label Jul 12, 2022
@ghost ghost requested review from dipeshmsft, singhashish-wpf and SamBent July 12, 2022 19:31
@ghost ghost added the Community Contribution A label for all community Contributions label Jul 12, 2022
@miloush
Copy link
Contributor

miloush commented Jul 28, 2022

Is there a reason for splitting the declaration from assignment?

@dipeshmsft dipeshmsft self-assigned this Aug 30, 2022
@dipeshmsft
Copy link
Member

Is there a reason for splitting the declaration from assignment?

Can't think of a reason, I guess we should merge the declaration and assignment for both the variables.

Copy link
Member

@dipeshmsft dipeshmsft left a comment

Choose a reason for hiding this comment

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

Can you merge the declaration here and remove the declarations from Line 241-242

@ghost ghost added the 📭 waiting-author-feedback To request more information from author. label Aug 31, 2022
@ghost ghost removed the 📭 waiting-author-feedback To request more information from author. label Sep 3, 2022
@VladimirKhil
Copy link
Contributor Author

Can you merge the declaration here and remove the declarations from Line 241-242

Done.

@dipeshmsft
Copy link
Member

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@pchaurasia14
Copy link
Member

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@pchaurasia14
Copy link
Member

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@dipeshmsft
Copy link
Member

Thanks for the contribution @VladimirKhil

singhashish-wpf pushed a commit that referenced this pull request Nov 1, 2022
…6785)

* Fix NullReferenceException in Invariant.get_IsDialogOverrideEnabled when .NET Framework Registry key (HKLM\Software\Microsoft\.NETFramework) is not present on the machine

#6772

* Update src/Microsoft.DotNet.Wpf/src/Shared/MS/Internal/Invariant.cs

Co-authored-by: Dipesh Kumar <[email protected]>

* Remove unnecessary declarations

Co-authored-by: Dipesh Kumar <[email protected]>
@ghost ghost locked as resolved and limited conversation to collaborators Nov 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Community Contribution A label for all community Contributions PR metadata: Label to tag PRs, to facilitate with triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MS.Internal.Invariant.get_IsDialogOverrideEnabled() can throw NullReferenceException
4 participants