store_chassis_index_if_needed() allocates entries while collecting
indices but does not destroy the temporary set.

Fixes: dae2eb8a17cc ("controller: support multiple chassis instances on the 
same host")
Signed-off-by: Ihar Hrachyshka <[email protected]>
Assisted-by: Codex, gpt-5.6-sol high
---
 controller/chassis.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/controller/chassis.c b/controller/chassis.c
index 09d15f5ef..be0332939 100644
--- a/controller/chassis.c
+++ b/controller/chassis.c
@@ -1155,8 +1155,9 @@ store_chassis_index_if_needed(
         /* All indices consumed: it's safer to just exit. */
         VLOG_ERR("All unique controller indices consumed. Exiting.");
         exit(EXIT_FAILURE);
-    }
 out:
+        sset_destroy(&used_indices);
+    }
     free(idx_key);
 }
 
-- 
2.54.0

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to