-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Open
Labels
PythonacknowledgedGitHub staff acknowledges this issueGitHub staff acknowledges this issuefalse-positivenot securityThis issue does not relate to a security queryThis issue does not relate to a security query
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
Metadata
Metadata
Assignees
Labels
PythonacknowledgedGitHub staff acknowledges this issueGitHub staff acknowledges this issuefalse-positivenot securityThis issue does not relate to a security queryThis issue does not relate to a security query

