Address static analysis issues
This patch is meant to address false-positive issues found by running the bandit static analysis tool. All the issues flagged were false positives, so the 'nosec' keyword has been added to the instances in order to allow bandit checks to pass. Change-Id: I8d16b7f8b2b3db574de210a322e3856075ead53c
This commit is contained in:
parent
6f668b4213
commit
dd8e64b6be
@ -26,7 +26,7 @@ from __future__ import print_function
|
||||
|
||||
import optparse
|
||||
import os
|
||||
import subprocess
|
||||
import subprocess # nosec
|
||||
import sys
|
||||
|
||||
|
||||
@ -61,7 +61,7 @@ class InstallVenv(object):
|
||||
else:
|
||||
stdout = None
|
||||
|
||||
proc = subprocess.Popen(cmd, cwd=self.root, stdout=stdout)
|
||||
proc = subprocess.Popen(cmd, cwd=self.root, stdout=stdout) # nosec
|
||||
output = proc.communicate()[0]
|
||||
if check_exit_code and proc.returncode != 0:
|
||||
self.die('Command "%s" failed.\n%s', ' '.join(cmd), output)
|
||||
|
Loading…
Reference in New Issue
Block a user