Hi Thomas, Qian has some problem with his email. So some patch can't be found in patch work. For it's a urgent bug. After we have aligned with Qian, Andy send this patch. Thanks.
> -----Original Message----- > From: Pei, Andy > Sent: Tuesday, April 23, 2019 9:19 > To: Thomas Monjalon <tho...@monjalon.net>; Xu, Rosen > <rosen...@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 > > 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? > >
