diff options
| author | Vadim Yanitskiy <axilirator@gmail.com> | 2018-03-06 02:11:06 +0700 |
|---|---|---|
| committer | Vadim Yanitskiy <axilirator@gmail.com> | 2018-03-06 02:28:37 +0700 |
| commit | c20f9bb5cd2491169959922ce1ba249a34f589aa (patch) | |
| tree | 67db26e9fa6e77c7eef9b04833ae7deecf23aec6 /src | |
| parent | 59d054906d1d5bd6f21fb6072ead0f76b6e5f27a (diff) | |
fake_trx/ctrl_if.py: remove incorrect isdigit() check
Previously, we used to check if all arguments of a command are
numeric. This was done in a wrong way, so parsing a *valid*
command with at least one negative argument could fail.
Let's remove this check, allowing the command handlers to
deal with argument types themselves.
Change-Id: If31295274a09102c414b5a7aec5dd85d88b2e514
Diffstat (limited to 'src')
| -rw-r--r-- | src/target/fake_trx/ctrl_if.py | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/target/fake_trx/ctrl_if.py b/src/target/fake_trx/ctrl_if.py index d4840cde..af124f65 100644 --- a/src/target/fake_trx/ctrl_if.py +++ b/src/target/fake_trx/ctrl_if.py @@ -60,11 +60,6 @@ class CTRLInterface(UDPLink): if len(request) - 1 != argc: return False - # Check if all arguments are numeric - for v in request[1:]: - if not v.isdigit(): - return False - return True def send_response(self, request, remote, response_code, params = None): |