github / github/codeql

LGTM.com - false positive - unnecessary null check C#

Open
#6,554 1 comment 0 reactions 0 assignees View on GitHub
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.