Re: [dm-devel] [PATCH 4/5] multipath: free vectors in configure

2021-05-12 Thread Benjamin Marzinski
On Wed, May 12, 2021 at 12:36:42PM +, Martin Wilck wrote: > On Tue, 2021-05-11 at 18:22 -0500, Benjamin Marzinski wrote: > > configure() can retry multiple times, each time reallocing a maps and > > paths vector, and leaking the previous ones. Fix this by always > > freeing > > the vectors befo

Re: [dm-devel] [PATCH 4/5] multipath: free vectors in configure

2021-05-12 Thread Martin Wilck
On Tue, 2021-05-11 at 18:22 -0500, Benjamin Marzinski wrote: > configure() can retry multiple times, each time reallocing a maps and > paths vector, and leaking the previous ones. Fix this by always > freeing > the vectors before configure() exits. Found by coverity. > > Signed-off-by: Benjamin Ma

[dm-devel] [PATCH 4/5] multipath: free vectors in configure

2021-05-11 Thread Benjamin Marzinski
configure() can retry multiple times, each time reallocing a maps and paths vector, and leaking the previous ones. Fix this by always freeing the vectors before configure() exits. Found by coverity. Signed-off-by: Benjamin Marzinski --- multipath/main.c | 7 ++- 1 file changed, 6 insertions(