Commit 45fbcb0d authored by Dong Yibo's avatar Dong Yibo
Browse files

RNPVF: NET: Fix "'%d' directive output may be truncated" warnings 

mucse inclusion
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/I9HCET


CVE: NA

--------------------------------

LKP reports below erros when building for loongarch64 with "W=1 build"

All warnings (new ones prefixed by >>):
---skip
drivers/net/ethernet/mucse/rnpvf/rnpvf_main.c:2128:25: note: 'snprintf'
	output between 10 and 29 bytes into a destination of size 24
2128 |                 snprintf(q_vector->name,
     |                 ^~~~~~~~~~~~~~~~~~~~~~~~
2129 |                       sizeof(q_vector->name) - 1, "%s-%s-%d-%d",
     |                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2130 |                       netdev->name, "TxRx", i, q_vector->v_idx);
     |                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
---skip

Fixes: 6adab536 (drivers: initial support for rnpvf drivers)
Reported-by: default avatarkernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202404030557.mZnvUtOv-lkp@intel.com/


Signed-off-by: default avatarDong Yibo <dong100@mucse.com>
parent 8c0dfc8b

drivers/net/ethernet/mucse/rnpvf/rnpvf_main.c

+4 −3
Original line number Diff line number Diff line
@@ -914,7 +914,8 @@ static void rnpvf_add_rx_frag(struct rnpvf_ring *rx_ring, 
 * @rx_ring: ring to place buffers on

 * @cleaned_count: number of buffers to replace

 **/

void rnpvf_alloc_rx_buffers(struct rnpvf_ring *rx_ring, u16 cleaned_count)

static void rnpvf_alloc_rx_buffers(struct rnpvf_ring *rx_ring,

				   u16 cleaned_count)

{

	union rnp_rx_desc *rx_desc;

	struct rnpvf_rx_buffer *bi;
@@ -2122,8 +2123,8 @@ static int rnpvf_request_msix_irqs(struct rnpvf_adapter *adapter) 


		if (q_vector->tx.ring && q_vector->rx.ring) {

			snprintf(q_vector->name,

				 sizeof(q_vector->name) - 1, "%s-%s-%d-%d",

				 netdev->name, "TxRx", i, q_vector->v_idx);

				 sizeof(q_vector->name), "%s-%s-%u",

				 netdev->name, "TxRx", i);

		} else {

			WARN(!(q_vector->tx.ring && q_vector->rx.ring),

			     "%s vector%d tx rx is null, v_idx:%d\n",