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:tho...@monjalon.net] Sent: Tuesday, April 23, 2019 1:52 AM To: Xu, Rosen <rosen...@intel.com>; Pei, Andy <andy....@intel.com> Cc: sta...@dpdk.org; dev@dpdk.org; Kovacevic, Marko <marko.kovace...@intel.com>; liq...@163.com 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: rosen...@intel.com > > Cc: marko.kovace...@intel.com > > Cc: liq...@163.com > > Cc: andy....@intel.com > > Cc: sta...@dpdk.org > > > > Signed-off-by: Andy Pei <andy....@intel.com> > > Acked-by: Rosen Xu <rosen...@intel.com> Li Qiang <liq...@163.com> proposed the same patch. Why he is not referenced here? Which patch should I merge?
