Re: [PATCH v2 2/2] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-04-05 Thread Rob Herring
On Tue, Apr 4, 2017 at 9:22 PM, jeffy wrote: > Hi Bjorn, > > > On 04/05/2017 03:18 AM, Bjorn Helgaas wrote: >> >> On Thu, Mar 23, 2017 at 5:58 PM, Dmitry Torokhov >> wrote: >>> >>> On Thu, Mar 23, 2017 at 3:07 PM, Rob Herring wrote:

Re: [PATCH v2 2/2] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-04-05 Thread Rob Herring
On Tue, Apr 4, 2017 at 9:22 PM, jeffy wrote: > Hi Bjorn, > > > On 04/05/2017 03:18 AM, Bjorn Helgaas wrote: >> >> On Thu, Mar 23, 2017 at 5:58 PM, Dmitry Torokhov >> wrote: >>> >>> On Thu, Mar 23, 2017 at 3:07 PM, Rob Herring wrote: On Thu, Mar 23, 2017 at 3:12 AM, Jeffy Chen

Re: [PATCH v2 2/2] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-04-04 Thread jeffy
Hi Bjorn, On 04/05/2017 03:18 AM, Bjorn Helgaas wrote: On Thu, Mar 23, 2017 at 5:58 PM, Dmitry Torokhov wrote: On Thu, Mar 23, 2017 at 3:07 PM, Rob Herring wrote: On Thu, Mar 23, 2017 at 3:12 AM, Jeffy Chen wrote: Currently we

Re: [PATCH v2 2/2] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-04-04 Thread jeffy
Hi Bjorn, On 04/05/2017 03:18 AM, Bjorn Helgaas wrote: On Thu, Mar 23, 2017 at 5:58 PM, Dmitry Torokhov wrote: On Thu, Mar 23, 2017 at 3:07 PM, Rob Herring wrote: On Thu, Mar 23, 2017 at 3:12 AM, Jeffy Chen wrote: Currently we only free the allocated resource struct when error. This would

Re: [PATCH v2 2/2] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-04-04 Thread Bjorn Helgaas
On Thu, Mar 23, 2017 at 5:58 PM, Dmitry Torokhov wrote: > On Thu, Mar 23, 2017 at 3:07 PM, Rob Herring wrote: >> On Thu, Mar 23, 2017 at 3:12 AM, Jeffy Chen >> wrote: >>> Currently we only free the allocated resource struct when

Re: [PATCH v2 2/2] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-04-04 Thread Bjorn Helgaas
On Thu, Mar 23, 2017 at 5:58 PM, Dmitry Torokhov wrote: > On Thu, Mar 23, 2017 at 3:07 PM, Rob Herring wrote: >> On Thu, Mar 23, 2017 at 3:12 AM, Jeffy Chen >> wrote: >>> Currently we only free the allocated resource struct when error. >>> This would cause memory leak after

Re: [PATCH v2 2/2] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-03-23 Thread jeffy
Hi Rob & Dmitry, On 03/24/2017 06:58 AM, Dmitry Torokhov wrote: On Thu, Mar 23, 2017 at 3:07 PM, Rob Herring wrote: On Thu, Mar 23, 2017 at 3:12 AM, Jeffy Chen wrote: Currently we only free the allocated resource struct when error. This would

Re: [PATCH v2 2/2] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-03-23 Thread jeffy
Hi Rob & Dmitry, On 03/24/2017 06:58 AM, Dmitry Torokhov wrote: On Thu, Mar 23, 2017 at 3:07 PM, Rob Herring wrote: On Thu, Mar 23, 2017 at 3:12 AM, Jeffy Chen wrote: Currently we only free the allocated resource struct when error. This would cause memory leak after pci_free_resource_list.

Re: [PATCH v2 2/2] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-03-23 Thread Dmitry Torokhov
On Thu, Mar 23, 2017 at 3:07 PM, Rob Herring wrote: > On Thu, Mar 23, 2017 at 3:12 AM, Jeffy Chen wrote: >> Currently we only free the allocated resource struct when error. >> This would cause memory leak after pci_free_resource_list. >> >>

Re: [PATCH v2 2/2] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-03-23 Thread Dmitry Torokhov
On Thu, Mar 23, 2017 at 3:07 PM, Rob Herring wrote: > On Thu, Mar 23, 2017 at 3:12 AM, Jeffy Chen wrote: >> Currently we only free the allocated resource struct when error. >> This would cause memory leak after pci_free_resource_list. >> >> Signed-off-by: Jeffy Chen >> --- >> >> Changes in v2:

Re: [PATCH v2 2/2] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-03-23 Thread Rob Herring
On Thu, Mar 23, 2017 at 3:12 AM, Jeffy Chen wrote: > Currently we only free the allocated resource struct when error. > This would cause memory leak after pci_free_resource_list. > > Signed-off-by: Jeffy Chen > --- > > Changes in v2: > Don't

Re: [PATCH v2 2/2] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-03-23 Thread Rob Herring
On Thu, Mar 23, 2017 at 3:12 AM, Jeffy Chen wrote: > Currently we only free the allocated resource struct when error. > This would cause memory leak after pci_free_resource_list. > > Signed-off-by: Jeffy Chen > --- > > Changes in v2: > Don't change the resource_list_create_entry's behavior. > >

[PATCH v2 2/2] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-03-23 Thread Jeffy Chen
Currently we only free the allocated resource struct when error. This would cause memory leak after pci_free_resource_list. Signed-off-by: Jeffy Chen --- Changes in v2: Don't change the resource_list_create_entry's behavior. drivers/of/of_pci.c | 57

[PATCH v2 2/2] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-03-23 Thread Jeffy Chen
Currently we only free the allocated resource struct when error. This would cause memory leak after pci_free_resource_list. Signed-off-by: Jeffy Chen --- Changes in v2: Don't change the resource_list_create_entry's behavior. drivers/of/of_pci.c | 57