Re: [PATCH V2] vhost-user-test: fix a memory leak

2020-01-16 Thread Pan Nengyuan
On 1/16/2020 9:29 PM, Thomas Huth wrote: > On 15/01/2020 10.13, Thomas Huth wrote: >> On 15/01/2020 04.10, Pan Nengyuan wrote: >>> >>> On 1/13/2020 10:32 AM, Pan Nengyuan wrote: On 1/12/2020 6:39 PM, Thomas Huth wrote: >> [...] > ... and now I had to unqueue the patch again. It is

Re: [PATCH V2] vhost-user-test: fix a memory leak

2020-01-16 Thread Thomas Huth
On 15/01/2020 10.13, Thomas Huth wrote: > On 15/01/2020 04.10, Pan Nengyuan wrote: >> >> On 1/13/2020 10:32 AM, Pan Nengyuan wrote: >>> >>> On 1/12/2020 6:39 PM, Thomas Huth wrote: > [...] ... and now I had to unqueue the patch again. It is reproducibly causing one of the gitlab CI pipeli

Re: [PATCH V2] vhost-user-test: fix a memory leak

2020-01-15 Thread Thomas Huth
On 15/01/2020 04.10, Pan Nengyuan wrote: > > On 1/13/2020 10:32 AM, Pan Nengyuan wrote: >> >> On 1/12/2020 6:39 PM, Thomas Huth wrote: [...] >>> ... and now I had to unqueue the patch again. It is reproducibly causing >>> one of the gitlab CI pipelines to fail with a timeout, e.g.: >>> >>> https:

Re: [PATCH V2] vhost-user-test: fix a memory leak

2020-01-14 Thread Pan Nengyuan
On 1/13/2020 10:32 AM, Pan Nengyuan wrote: > > > On 1/12/2020 6:39 PM, Thomas Huth wrote: >> On 10/01/2020 15.07, Thomas Huth wrote: >>> On 20/12/2019 02.26, pannengy...@huawei.com wrote: From: Pan Nengyuan Spotted by ASAN. Reported-by: Euler Robot Signed-off-by

Re: [PATCH V2] vhost-user-test: fix a memory leak

2020-01-12 Thread Pan Nengyuan
On 1/12/2020 6:39 PM, Thomas Huth wrote: > On 10/01/2020 15.07, Thomas Huth wrote: >> On 20/12/2019 02.26, pannengy...@huawei.com wrote: >>> From: Pan Nengyuan >>> >>> Spotted by ASAN. >>> >>> Reported-by: Euler Robot >>> Signed-off-by: Pan Nengyuan >>> --- >>> Changes V2 to V1: >>> - use a "

Re: [PATCH V2] vhost-user-test: fix a memory leak

2020-01-12 Thread Thomas Huth
On 10/01/2020 15.07, Thomas Huth wrote: > On 20/12/2019 02.26, pannengy...@huawei.com wrote: >> From: Pan Nengyuan >> >> Spotted by ASAN. >> >> Reported-by: Euler Robot >> Signed-off-by: Pan Nengyuan >> --- >> Changes V2 to V1: >> - use a "goto cleanup", instead of duplicating the "free" functio

Re: [PATCH V2] vhost-user-test: fix a memory leak

2020-01-10 Thread Thomas Huth
On 20/12/2019 02.26, pannengy...@huawei.com wrote: > From: Pan Nengyuan > > Spotted by ASAN. > > Reported-by: Euler Robot > Signed-off-by: Pan Nengyuan > --- > Changes V2 to V1: > - use a "goto cleanup", instead of duplicating the "free" functions. > - free "dest_cmdline" at the end. > --- >

[PATCH V2] vhost-user-test: fix a memory leak

2019-12-19 Thread pannengyuan
From: Pan Nengyuan Spotted by ASAN. Reported-by: Euler Robot Signed-off-by: Pan Nengyuan --- Changes V2 to V1: - use a "goto cleanup", instead of duplicating the "free" functions. - free "dest_cmdline" at the end. --- tests/vhost-user-test.c | 5 - 1 file changed, 4 insertions(+), 1 delet