On 9/21/2018 5:25 PM, Paul Luse wrote: > From: paul luse <paul.e.l...@intel.com> > > This patch fixes an issue caught with ASAN where a vdev_scan() > to a secondary bus was failing to free some memory. > > Signed-off-by: paul luse <paul.e.l...@intel.com>
Hi Paul, Is this different from below patch, which seems already acked: https://patches.dpdk.org/patch/45079/ > --- > drivers/bus/vdev/vdev.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/bus/vdev/vdev.c b/drivers/bus/vdev/vdev.c > index 69dee89a8..2566d6c0d 100644 > --- a/drivers/bus/vdev/vdev.c > +++ b/drivers/bus/vdev/vdev.c > @@ -423,6 +423,7 @@ vdev_scan(void) > mp_rep = &mp_reply.msgs[0]; > resp = (struct vdev_param *)mp_rep->param; > VDEV_LOG(INFO, "Received %d vdevs", resp->num); > + free(mp_reply.msgs); > } else > VDEV_LOG(ERR, "Failed to request vdev from primary"); > >
