Merge pull request #14467 from cscarpitta/bugfix/fix-srv6-isis-memleaks

isisd: Fix memory leaks when IS-IS fails to process an SRv6 locator chunk
FRRouting · Sep 24, 2023 · 1c829fa · 1c829fa
2 parents 56d8305 + 46d1028
commit 1c829fa
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/isisd/isis_zebra.c b/isisd/isis_zebra.c
@@ -1136,13 +1136,17 @@ static int isis_zebra_process_srv6_locator_chunk(ZAPI_CALLBACK_ARGS)
 	enum srv6_endpoint_behavior_codepoint behavior;
 	bool allocated = false;
 
-	if (!isis)
+	if (!isis) {
+		srv6_locator_chunk_free(&chunk);
 		return -1;
+	}
 
 	/* Decode the received zebra message */
 	s = zclient->ibuf;
-	if (zapi_srv6_locator_chunk_decode(s, chunk) < 0)
+	if (zapi_srv6_locator_chunk_decode(s, chunk) < 0) {
+		srv6_locator_chunk_free(&chunk);
 		return -1;
+	}
 
 	sr_debug(
 		"Received SRv6 locator chunk from zebra: name %s, "