> -----Original Message----- > From: stable <[email protected]> On Behalf Of Qiming Chen > Sent: Friday, September 10, 2021 11:13 > To: [email protected] > Cc: Xing, Beilei <[email protected]>; Wu, Jingjing > <[email protected]>; Qiming Chen > <[email protected]>; [email protected] > Subject: [dpdk-stable] [PATCH] common/iavf: fix arq resource leak > > In the iavf_init_arq function, if an exception occurs in the > iavf_config_arq_regs function, and the previously applied arq bufs > resource is released. This patch maintains the same modification > as the iavf_init_asq function to roll back resources. > > Fixes: 87aca6d8d8a4 ("net/iavf/base: fix command buffer memory leak") > Cc: [email protected] > > Signed-off-by: Qiming Chen <[email protected]> > --- > drivers/common/iavf/iavf_adminq.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) >
Can't image that you ARE superman to cover ixgbe,ixgbevf, i40e, now iavf. It MUST BE done by some code scan tool, I don't believe you have real time to read the whole code. Right ? Sometimes, Defensive programming is good, but ... I prefer to the PMD can run well every time, if failed, it must have some bugs. > -- > 2.30.1.windows.1
