summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorIWASE Yusuke <iwase.yusuke0@gmail.com>2016-12-16 15:17:38 +0900
committerFUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>2016-12-29 22:58:23 +0900
commit7e30cf6b0136b97823e8a17280d69eb482ca1fa7 (patch)
tree26b54dbeeb45fd060f4780b87727c8e5bc5b67db
parent98a3b03792879741d4eb0f2f52f9d55ec64cea3d (diff)
rpc_cli: Avoid to use eval()
Signed-off-by: IWASE Yusuke <iwase.yusuke0@gmail.com> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
-rwxr-xr-xryu/cmd/rpc_cli.py7
1 files changed, 4 insertions, 3 deletions
diff --git a/ryu/cmd/rpc_cli.py b/ryu/cmd/rpc_cli.py
index 2c1eb6b3..25b92fa5 100755
--- a/ryu/cmd/rpc_cli.py
+++ b/ryu/cmd/rpc_cli.py
@@ -31,6 +31,7 @@
from __future__ import print_function
+import ast
import cmd
import signal
import socket
@@ -139,9 +140,9 @@ class Cmd(cmd.Cmd):
try:
peer = args[0]
method = args[1]
- params = eval(args[2])
- except:
- print("argument error")
+ params = ast.literal_eval(args[2])
+ except (IndexError, ValueError) as e:
+ print("argument error: %s" % e)
return
try:
p = peers[peer]