Hi Thomas, My patch is the same as LI Qiang's patch. I was told that Qiang's patch cannot get onto the patchwork, so I just help him do this.
-----Original Message----- From: Thomas Monjalon [mailto:[email protected]] Sent: Tuesday, April 23, 2019 1:52 AM To: Xu, Rosen <[email protected]>; Pei, Andy <[email protected]> Cc: [email protected]; [email protected]; Kovacevic, Marko <[email protected]>; [email protected] Subject: Re: [dpdk-stable] [PATCH v2] raw/ifpga_rawdev: fix fd leak in rte fpga do pr > > In rte_fpga_do_pr() function, if 'stat' returns error, > > rte_fpga_do_pr() returns -EINVAL without closing the 'file_fd' that > > has been opened. > > After this patch, 'file_fd' is closed before rte_fpga_do_pr() > > returns -EINVAL when 'stat' returns error > > > > Coverity issue: 279441 > > Fixes: ef1e8ede3da5 ("raw/ifpga: add Intel FPGA bus rawdev driver") > > Cc: [email protected] > > Cc: [email protected] > > Cc: [email protected] > > Cc: [email protected] > > Cc: [email protected] > > > > Signed-off-by: Andy Pei <[email protected]> > > Acked-by: Rosen Xu <[email protected]> Li Qiang <[email protected]> proposed the same patch. Why he is not referenced here? Which patch should I merge?
