LGTM.com - false positive with null checks in C#
Open
C#
false-positive
- Dominant language
- CodeQL
- Stars
- 10.1k
- Forks
- 2.1k
- Avg merge
- 2d 15h
- Merged PRs (30d)
- 141
Description
A null check and throw in C# before a sensitive method is detected as an error. This should not result in an error, as removing it would then make an NRE happen when we deref, which is much worse for debugging
https://lgtm.com/projects/g/microsoft/msquic/snapshot/9a3f71038c6dc2b1c80a6b59e9f43bcd92ea4254/files/src/perf/dbserver/Controllers/PerformanceController.cs?sort=name&dir=ASC&mode=heatmap#xfd2d1b7771fa1935:1
Contributor guide
Assessment
This issue has not been assessed yet.