From: Andrius Sirvys Date: Thu, 8 Aug 2019 14:37:52 +0000 (+0100) Subject: usertools: fix input handling in telemetry script X-Git-Url: http://git.droids-corp.org/?p=dpdk.git;a=commitdiff_plain;h=a667070b09bb8cfbd83912045391d877ed88725e usertools: fix input handling in telemetry script This commit removes the unnecesarry ast.literal_eval() function call from the input handling, which now relies just on raw_input() to get its input. Fixes: 53f293c9a783 ("usertools: replace unsafe input function") Cc: stable@dpdk.org Signed-off-by: Andrius Sirvys Acked-by: Harry van Haaren --- diff --git a/usertools/dpdk-telemetry-client.py b/usertools/dpdk-telemetry-client.py index 4b0502ff90..60fe97af79 100755 --- a/usertools/dpdk-telemetry-client.py +++ b/usertools/dpdk-telemetry-client.py @@ -78,7 +78,7 @@ class 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(ast.literal_eval(raw_input("\n:"))) + n_requests = int(raw_input("\n:")) print("\033[F") #Removes the user input from screen, cleans it up print("\033[K") for i in range(n_requests): @@ -99,7 +99,7 @@ class Client: print("[4] Unregister client") try: - self.choice = int(ast.literal_eval(raw_input("\n:"))) + self.choice = int(raw_input("\n:")) print("\033[F") #Removes the user input for screen, cleans it up print("\033[K") if self.choice == 1: