Re: [dpdk-dev] [PATCH 1/3] bus/vmbus: fix leak on device scan

2021-10-01 Thread Long Li
> Subject: Re: [PATCH 1/3] bus/vmbus: fix leak on device scan > > On Thu, Sep 30, 2021 at 9:14 PM Long Li wrote: > > > rte_device name only points at some location where the name is stored. > > > In general this storage is in the bus object or (in some buses) the > > > devarg that resulted in the

Re: [dpdk-dev] [PATCH 1/3] bus/vmbus: fix leak on device scan

2021-09-30 Thread David Marchand
On Thu, Sep 30, 2021 at 9:14 PM Long Li wrote: > > rte_device name only points at some location where the name is stored. > > In general this storage is in the bus object or (in some buses) the devarg > > that > > resulted in the rte_device object creation. > > > > If we won't store the name in t

Re: [dpdk-dev] [PATCH 1/3] bus/vmbus: fix leak on device scan

2021-09-30 Thread Long Li
> Subject: Re: [PATCH 1/3] bus/vmbus: fix leak on device scan > > On Wed, Sep 29, 2021 at 10:57 PM Long Li wrote: > > > > > Subject: [PATCH 1/3] bus/vmbus: fix leak on device scan > > > > > > Caught running ASAN. > > > > > > The device name is leaked on scan. > > > rte_device name field being a c

Re: [dpdk-dev] [PATCH 1/3] bus/vmbus: fix leak on device scan

2021-09-30 Thread David Marchand
On Wed, Sep 29, 2021 at 10:57 PM Long Li wrote: > > > Subject: [PATCH 1/3] bus/vmbus: fix leak on device scan > > > > Caught running ASAN. > > > > The device name is leaked on scan. > > rte_device name field being a const, use the private vmbus struct to store > > the > > device name and point at

Re: [dpdk-dev] [PATCH 1/3] bus/vmbus: fix leak on device scan

2021-09-29 Thread Long Li
> Subject: [PATCH 1/3] bus/vmbus: fix leak on device scan > > Caught running ASAN. > > The device name is leaked on scan. > rte_device name field being a const, use the private vmbus struct to store the > device name and point at it. > > Fixes: 831dba47bd36 ("bus/vmbus: add Hyper-V virtual bus s

[dpdk-dev] [PATCH 1/3] bus/vmbus: fix leak on device scan

2021-09-17 Thread David Marchand
Caught running ASAN. The device name is leaked on scan. rte_device name field being a const, use the private vmbus struct to store the device name and point at it. Fixes: 831dba47bd36 ("bus/vmbus: add Hyper-V virtual bus support") Cc: sta...@dpdk.org Signed-off-by: David Marchand --- drivers/b