diff mbox series

[1/2] libmultipath: free allocated value in set_int

Message ID 1539290628-16948-1-git-send-email-bmarzins@redhat.com (mailing list archive)
State Not Applicable, archived
Delegated to: christophe varoqui
Headers show
Series [1/2] libmultipath: free allocated value in set_int | expand

Commit Message

Benjamin Marzinski Oct. 11, 2018, 8:43 p.m. UTC
set_int() needs to free the buffer it got back from set_value()

Fixes: 8f9ac30d ("libmultipath: fix set_int error path")
Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
---
 libmultipath/dict.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Martin Wilck Oct. 11, 2018, 9:46 p.m. UTC | #1
On Thu, 2018-10-11 at 15:43 -0500, Benjamin Marzinski wrote:
> set_int() needs to free the buffer it got back from set_value()
> 
> Fixes: 8f9ac30d ("libmultipath: fix set_int error path")
> Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>

Reviewed-by: Martin Wilck <mwilck@suse.com>
diff mbox series

Patch

diff --git a/libmultipath/dict.c b/libmultipath/dict.c
index bf4701e..c3f5a6e 100644
--- a/libmultipath/dict.c
+++ b/libmultipath/dict.c
@@ -39,6 +39,7 @@  set_int(vector strvec, void *ptr)
 
 	*int_ptr = atoi(buff);
 
+	FREE(buff);
 	return 0;
 }