> +static int
> +security_handle_cryptodev_crypto_caps(const char *cmd __rte_unused,
> const char *params,
> + struct rte_tel_data *d)
> +{
> + const struct rte_security_capability *capabilities;
> + struct rte_tel_data *crypto_caps;
> + const char *capa_param;
> + int dev_id, capa_id;
> + int crypto_caps_n;
> + char *end_param;
> + int rc;
> +
> + if (!params || strlen(params) == 0 || !isdigit(*params))
> + return -EINVAL;
> +
> + dev_id = strtoul(params, &end_param, 0);
> + capa_param = strtok(end_param, ",");
> + if (!capa_param || strlen(capa_param) == 0 ||
> !isdigit(*capa_param))
> + return -EINVAL;
> +
> + capa_id = strtoul(capa_param, &end_param, 0);
> + if (*end_param != '\0')
> + CDEV_LOG_ERR("Extra parameters passed to command,
> ignoring");
> +
> + rc = security_capabilities_from_dev_id(dev_id, (void *)&capabilities);
> + if (rc < 0)
> + return rc;
> +
> + crypto_caps = rte_tel_data_alloc();
> + RTE_PTR_OR_ERR_RET(crypto_caps, -ENOMEM);
> +
> + rte_tel_data_start_dict(d);
> + crypto_caps_n = crypto_caps_array(crypto_caps, capabilities-
> >crypto_capabilities);
> + if (capa_id >= crypto_caps_n) {
> + CDEV_LOG_ERR("Extra parameters passed to command,
> ignoring");
> + return -EINVAL;
> + }
Something is not correct here.
Capa_id is not getting used properly.
Security_capabilities should be traversed until capa_id and then extract
The corresponding crypto capabilities. Right?
> +
> + rte_tel_data_add_dict_container(d, "crypto_caps", crypto_caps, 0);
> + rte_tel_data_add_dict_int(d, "crypto_caps_n", crypto_caps_n);
> +
> + return 0;
> +}
> +
> +RTE_INIT(security_init_telemetry)
> +{
> + rte_telemetry_register_cmd("/security/cryptodev/list",
> + security_handle_cryptodev_list,
> + "Returns list of available crypto devices by IDs. No
> parameters.");
> +
> + rte_telemetry_register_cmd("/security/cryptodev/sec_caps",
> + security_handle_cryptodev_sec_caps,
> + "Returns security capabilities for a cryptodev. Parameters: int
> dev_id");
> +
> + rte_telemetry_register_cmd("/security/cryptodev/crypto_caps",
> + security_handle_cryptodev_crypto_caps,
> + "Returns crypto capabilities for a security capability.
> Parameters: int dev_id, sec_cap_id");
> +}
> --
> 2.25.1
