LGTM.com - false positive - unnecessary null check C#
Open
acknowledged
C#
false-positive
not security
- Dominant language
- CodeQL
- Stars
- 10.1k
- Forks
- 2.1k
- Avg merge
- 2d 15h
- Merged PRs (30d)
- 141
Description
Found it in a **.NET 5.0 C#** project of mine, an access to a **List\** object in a loop had a warning saying it may be null because of some checks.
This shouldn't be the case because the loop is already limited by a variable **n** which gets **0** in the case of the object being null, or the length of the object. When the **n** variable is **0** loop is skipped, therefore the null check inside it is unnecessary.
[**>>Alert link here<<**](https://lgtm.com/projects/g/semihM/exmat/snapshot/094b2dcf4a99b7e555bdb418d990a04b9344df98/files/exmatstdlib/ExStdString.cs?sort=name&dir=ASC&mode=heatmap#xf95a12ccdeb2d93c:1)
Contributor guide
Assessment
This issue has not been assessed yet.