[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v2] xen: fix memory leak in __xen_pcibk_add_pci_dev()
>>> On 01.04.14 at 12:15, <daeseok.youn@xxxxxxxxx> wrote: > It need to free dev_entry when it failed to assign to a new > slot on the virtual PCI bus. > > smatch says: > drivers/xen/xen-pciback/vpci.c:142 __xen_pcibk_add_pci_dev() warn: > possible memory leak of 'dev_entry' > > Signed-off-by: Daeseok Youn <daeseok.youn@xxxxxxxxx> As before Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx> > --- > v2: The kfree() invocation is moved outside the locked region. > > drivers/xen/xen-pciback/vpci.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/drivers/xen/xen-pciback/vpci.c b/drivers/xen/xen-pciback/vpci.c > index 3165ce3..51afff9 100644 > --- a/drivers/xen/xen-pciback/vpci.c > +++ b/drivers/xen/xen-pciback/vpci.c > @@ -137,6 +137,8 @@ unlock: > /* Publish this device. */ > if (!err) > err = publish_cb(pdev, 0, 0, PCI_DEVFN(slot, func), devid); > + else > + kfree(dev_entry); > > out: > return err; > -- > 1.7.4.4 _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |