Re: [PATCH v2 00/17] drivers: replace strncpy with strlcpy
Bruce Richardson <[email protected]>
| Newsgroups | org.dpdk.dev |
|---|---|
| Message-ID | <[email protected]> |
Driver maintainers, ping for review please! Thanks, /Bruce On Wed, Jun 24, 2026 at 11:36:39AM +0100, Bruce Richardson wrote: > Many uses of strncpy in DPDK drivers can be directly replaced by > a call to strlcpy instead, which is safer in that it always null- > terminates the string. This AI assisted patchset makes those > simple replacements, adjusting lengths as appropriate. > > After this set, there are still a number of drivers with strncpy calls > in them, but those are not simple strncpy->strlcpy replacements, so > left for later rework. > > v2: > * fix incorrect commit ids for fixlines. > * fix build errors due to missing rte_string_fns.h header include > > Bruce Richardson (17): > common/cnxk: replace strncpy with strlcpy > common/mlx5: replace strncpy with strlcpy > crypto/mvsam: replace strncpy with strlcpy > crypto/scheduler: replace strncpy with strlcpy > event/cnxk: replace strncpy with strlcpy > net/ark: replace strncpy with strlcpy > net/bnx2x: replace strncpy with strlcpy > net/cnxk: replace strncpy with strlcpy > net/dpaa: replace strncpy with strlcpy > net/ionic: replace strncpy with strlcpy > net/mlx4: replace strncpy with strlcpy > net/mlx5: replace strncpy with strlcpy > net/nfp: replace strncpy with strlcpy > net/qede: replace strncpy with strlcpy > net/softnic: replace strncpy with strlcpy > net/vhost: replace strncpy with strlcpy > net/virtio: replace strncpy with strlcpy > > drivers/common/cnxk/roc_model.c | 8 ++++---- > drivers/common/mlx5/linux/mlx5_common_os.c | 2 +- > drivers/crypto/mvsam/rte_mrvl_pmd.c | 2 +- > drivers/crypto/scheduler/scheduler_pmd.c | 8 ++++---- > drivers/event/cnxk/cnxk_eventdev_stats.c | 2 +- > drivers/net/ark/ark_ethdev.c | 2 +- > drivers/net/bnx2x/bnx2x.c | 2 +- > drivers/net/bnx2x/bnx2x_vfpf.c | 2 +- > drivers/net/cnxk/cnxk_eswitch.c | 4 ++-- > drivers/net/cnxk/cnxk_rep_msg.c | 2 +- > drivers/net/dpaa/dpaa_ethdev.c | 2 +- > drivers/net/ionic/ionic_main.c | 4 ++-- > drivers/net/mlx4/mlx4_ethdev.c | 2 +- > drivers/net/mlx5/mlx5.c | 8 ++++---- > drivers/net/mlx5/windows/mlx5_ethdev_os.c | 2 +- > drivers/net/nfp/nfpcore/nfp_resource.c | 2 +- > drivers/net/qede/qede_ethdev.c | 2 +- > drivers/net/softnic/conn.c | 6 ++++-- > drivers/net/vhost/rte_eth_vhost.c | 2 +- > drivers/net/virtio/virtio_user/vhost_kernel_tap.c | 2 +- > 20 files changed, 34 insertions(+), 32 deletions(-) > > -- > 2.53.0 >