diff options
author | IWASE Yusuke <iwase.yusuke0@gmail.com> | 2016-12-16 15:17:38 +0900 |
---|---|---|
committer | FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> | 2016-12-29 22:58:23 +0900 |
commit | 7e30cf6b0136b97823e8a17280d69eb482ca1fa7 (patch) | |
tree | 26b54dbeeb45fd060f4780b87727c8e5bc5b67db | |
parent | 98a3b03792879741d4eb0f2f52f9d55ec64cea3d (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-x | ryu/cmd/rpc_cli.py | 7 |
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] |