Re: [Qemu-devel] [PATCH] piix: fix resource leak reported by Coverity

2015-10-29 Thread Michael Tokarev
14.09.2015 13:40, zhanghailiang wrote: > config_fd should be closed before return, or there will > be a resource leak error. > > Signed-off-by: zhanghailiang > --- > hw/pci-host/piix.c | 11 +++ > 1 file changed, 7 insertions(+), 4 deletions(-) > > diff --git a/hw/pci-host/piix.c b/hw/p

Re: [Qemu-devel] [PATCH] piix: fix resource leak reported by Coverity

2015-10-16 Thread Stefan Hajnoczi
On Mon, Sep 14, 2015 at 06:40:10PM +0800, zhanghailiang wrote: > config_fd should be closed before return, or there will > be a resource leak error. > > Signed-off-by: zhanghailiang > --- > hw/pci-host/piix.c | 11 +++ > 1 file changed, 7 insertions(+), 4 deletions(-) Reviewed-by: Stefa

[Qemu-devel] [PATCH] piix: fix resource leak reported by Coverity

2015-09-14 Thread zhanghailiang
config_fd should be closed before return, or there will be a resource leak error. Signed-off-by: zhanghailiang --- hw/pci-host/piix.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/hw/pci-host/piix.c b/hw/pci-host/piix.c index 1fb71c8..7b2fbf9 100644 --- a/hw/pci