Skip to content

Commit

Permalink
usertools: replace unsafe input function
Browse files Browse the repository at this point in the history
[ upstream commit 53f293c ]

LGTM static code analysis tool reports that the function 'input' is
unsafe. Changed to use raw_input which then converts it using
ast.literal_eval() which is safe.

Fixes: d1b94da ("usertools: add client script for telemetry")

Signed-off-by: Andrius Sirvys <andrius.sirvys@intel.com>
Acked-by: Kevin Laatz <kevin.laatz@intel.com>
  • Loading branch information
Andrius Sirvys authored and kevintraynor committed Sep 4, 2019
1 parent 784e0fe commit 5636553
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions usertools/dpdk-telemetry-client.py
Expand Up @@ -13,6 +13,11 @@
API_UNREG = "{\"action\":2,\"command\":\"clients\",\"data\":{\"client_path\":\""
DEFAULT_FP = "/var/run/dpdk/default_client"

try:
raw_input # Python 2
except NameError:
raw_input = input # Python 3

class Socket:

def __init__(self):
Expand Down Expand Up @@ -71,7 +76,7 @@ def requestMetrics(self): # Requests metrics for given client

def repeatedlyRequestMetrics(self, sleep_time): # Recursively requests metrics for given client
print("\nPlease enter the number of times you'd like to continuously request Metrics:")
n_requests = int(input("\n:"))
n_requests = int(ast.literal_eval(raw_input("\n:")))
print("\033[F") #Removes the user input from screen, cleans it up
print("\033[K")
for i in range(n_requests):
Expand All @@ -86,7 +91,7 @@ def interactiveMenu(self, sleep_time): # Creates Interactive menu within the scr
print("[3] Unregister client")

try:
self.choice = int(input("\n:"))
self.choice = int(ast.literal_eval(raw_input("\n:")))
print("\033[F") #Removes the user input for screen, cleans it up
print("\033[K")
if self.choice == 1:
Expand Down

0 comments on commit 5636553

Please sign in to comment.