Replace snprintf with strlcpy where possible. Signed-off-by: Stephen Hemminger <step...@networkplumber.org> --- lib/bbdev/rte_bbdev.c | 2 +- lib/ethdev/sff_8636.c | 11 +++++++---- 2 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/lib/bbdev/rte_bbdev.c b/lib/bbdev/rte_bbdev.c index 581509b5a213..7cc126a0e7be 100644 --- a/lib/bbdev/rte_bbdev.c +++ b/lib/bbdev/rte_bbdev.c @@ -212,7 +212,7 @@ rte_bbdev_allocate(const char *name) bbdev->data->dev_id = dev_id; bbdev->state = RTE_BBDEV_INITIALIZED; - ret = snprintf(bbdev->data->name, RTE_BBDEV_NAME_MAX_LEN, "%s", name); + ret = strlcpy(bbdev->data->name, name, RTE_BBDEV_NAME_MAX_LEN); if ((ret < 0) || (ret >= RTE_BBDEV_NAME_MAX_LEN)) { rte_bbdev_log(ERR, "Copying device name \"%s\" failed", name); return NULL; diff --git a/lib/ethdev/sff_8636.c b/lib/ethdev/sff_8636.c index 6b65f47efedf..a623e0d065d8 100644 --- a/lib/ethdev/sff_8636.c +++ b/lib/ethdev/sff_8636.c @@ -203,16 +203,19 @@ static void sff_8636_show_ext_identifier(const uint8_t *data, struct rte_tel_dat switch (data[SFF_8636_EXT_ID_OFFSET] & SFF_8636_EXT_ID_EPWR_CLASS_MASK) { case SFF_8636_EXT_ID_PWR_CLASS_LEGACY: - snprintf(val_string, sizeof(val_string), "%s", ""); + strlcpy(val_string, "", sizeof(val_string)); break; case SFF_8636_EXT_ID_PWR_CLASS_5: - snprintf(val_string, sizeof(val_string), "%s", "4.0W max. Power consumption, "); + strlcpy(val_string, "4.0W max. Power consumption, ", + sizeof(val_string)); break; case SFF_8636_EXT_ID_PWR_CLASS_6: - snprintf(val_string, sizeof(val_string), "%s", "4.5W max. Power consumption, "); + strlcpy(val_string, "4.5W max. Power consumption, ", + sizeof(val_string)); break; case SFF_8636_EXT_ID_PWR_CLASS_7: - snprintf(val_string, sizeof(val_string), "%s", "5.0W max. Power consumption, "); + strlcpy(val_string, "5.0W max. Power consumption, ", + sizeof(val_string)); break; } -- 2.39.2