github / github/codeql

LGTM.com - false positive (captured variable)

Ouverte
#6,457 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
acknowledged false-positive not security Python
Langage dominant
CodeQL
Étoiles
10.1k
Forks
2.1k
Merge moyen
2 j 15 h
PR mergées (30 j)
141

Description

**Description of the false positive**

A variable is defined inside a function, and later used/access from a nested function as follows:

```
def manage_dvs(request) -> str:
...
curr_fake_platform = None
...
def update_dvs(log_path, new_fake_platform=None):
...
nonlocal curr_fake_platform
if new_fake_platform != curr_fake_platform:
...
```

The `curr_fake_platform` incorrectly triggers an unused variable alert

**URL to the alert on the project page on LGTM.com**
https://lgtm.com/projects/g/Azure/sonic-swss/snapshot/b5ac9505d8ac0a7830488d825aae7e616480cc86/files/tests/conftest.py?sort=name&dir=ASC&mode=heatmap#xd660241c258f5ae6:1

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.