diff mbox series

[BlueZ] mesh: Fix potential memory leak

Message ID 20220630211313.210017-1-brian.gix@intel.com
State New
Headers show
Series [BlueZ] mesh: Fix potential memory leak | expand

Commit Message

Brian Gix June 30, 2022, 9:13 p.m. UTC 
This memory leak will never happen, however since we added a new
return from function that malloc'd memory, the free should still be
done.
---
 mesh/mesh-config-json.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/mesh/mesh-config-json.c b/mesh/mesh-config-json.c
index 5bb1e5ce0..7f46c8582 100644
--- a/mesh/mesh-config-json.c
+++ b/mesh/mesh-config-json.c
@@ -1961,8 +1961,10 @@  bool mesh_config_comp_page_add(struct mesh_config *cfg, uint8_t page,
 	len = (size * 2) + 3;
 	buf = l_malloc(len);
 	ret = snprintf(buf, len, "%2.2x", page);
-	if (ret < 0)
+	if (ret < 0) {
+		l_free(buf);
 		return false;
+	}
 
 	hex2str(data, size, buf + 2, len - 2);