From 5514319e7b437a6a252e84d7d67d728f3082cd97 Mon Sep 17 00:00:00 2001 From: Ciara Power Date: Thu, 27 Aug 2020 09:39:22 +0100 Subject: [PATCH] telemetry: fix passing full params string to command Telemetry only passed the first param to the command handler if multiple were entered by the user, separated by commas. Telemetry is required to pass the full params string to the command, by splitting by a comma delimiter only once to remove the command part of the string. This will enable future commands to take multiple param values. Fixes: b1ad0e124536 ("rawdev: add telemetry callbacks") Fixes: c190daedb9b1 ("ethdev: add telemetry callbacks") Fixes: 6dd571fd07c3 ("telemetry: introduce new functionality") Cc: stable@dpdk.org Signed-off-by: Ciara Power Acked-by: Bruce Richardson --- lib/librte_ethdev/rte_ethdev.c | 12 ++++++++++-- lib/librte_rawdev/rte_rawdev.c | 6 +++++- lib/librte_telemetry/telemetry.c | 2 +- 3 files changed, 16 insertions(+), 4 deletions(-) diff --git a/lib/librte_ethdev/rte_ethdev.c b/lib/librte_ethdev/rte_ethdev.c index d7668114ca..f1e44594c6 100644 --- a/lib/librte_ethdev/rte_ethdev.c +++ b/lib/librte_ethdev/rte_ethdev.c @@ -5324,11 +5324,15 @@ handle_port_xstats(const char *cmd __rte_unused, struct rte_eth_xstat_name *xstat_names; int port_id, num_xstats; int i, ret; + char *end_param; if (params == NULL || strlen(params) == 0 || !isdigit(*params)) return -1; - port_id = atoi(params); + port_id = strtoul(params, &end_param, 0); + if (*end_param != '\0') + RTE_ETHDEV_LOG(NOTICE, + "Extra parameters passed to ethdev telemetry command, ignoring"); if (!rte_eth_dev_is_valid_port(port_id)) return -1; @@ -5370,11 +5374,15 @@ handle_port_link_status(const char *cmd __rte_unused, static const char *status_str = "status"; int ret, port_id; struct rte_eth_link link; + char *end_param; if (params == NULL || strlen(params) == 0 || !isdigit(*params)) return -1; - port_id = atoi(params); + port_id = strtoul(params, &end_param, 0); + if (*end_param != '\0') + RTE_ETHDEV_LOG(NOTICE, + "Extra parameters passed to ethdev telemetry command, ignoring"); if (!rte_eth_dev_is_valid_port(port_id)) return -1; diff --git a/lib/librte_rawdev/rte_rawdev.c b/lib/librte_rawdev/rte_rawdev.c index 36f3acf2b7..f29164dd15 100644 --- a/lib/librte_rawdev/rte_rawdev.c +++ b/lib/librte_rawdev/rte_rawdev.c @@ -578,11 +578,15 @@ handle_dev_xstats(const char *cmd __rte_unused, struct rte_rawdev_xstats_name *xstat_names; int dev_id, num_xstats, i, ret; unsigned int *ids; + char *end_param; if (params == NULL || strlen(params) == 0 || !isdigit(*params)) return -1; - dev_id = atoi(params); + dev_id = strtoul(params, &end_param, 0); + if (*end_param != '\0') + RTE_RDEV_LOG(NOTICE, + "Extra parameters passed to rawdev telemetry command, ignoring"); if (!rte_rawdev_pmd_is_valid_dev(dev_id)) return -1; diff --git a/lib/librte_telemetry/telemetry.c b/lib/librte_telemetry/telemetry.c index 51e7ceeb1b..78754050a5 100644 --- a/lib/librte_telemetry/telemetry.c +++ b/lib/librte_telemetry/telemetry.c @@ -258,7 +258,7 @@ client_handler(void *sock_id) while (bytes > 0) { buffer[bytes] = 0; const char *cmd = strtok(buffer, ","); - const char *param = strtok(NULL, ","); + const char *param = strtok(NULL, "\0"); telemetry_cb fn = unknown_command; int i; -- 2.20.1