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 <[email protected]>; Xu, Rosen > <[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 > > 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? > >
