For patches 1 and 2:

Acked-by: Mark Michelson <[email protected]>

Given their simplicity and how obviously correct they are, I went ahead and merged these to main, branch-22.06, branch-22.03, and branch-21.12.

On 6/24/22 18:41, Ihar Hrachyshka wrote:
The leak is reported for asan runs.

Signed-off-by: Ihar Hrachyshka <[email protected]>
---
  utilities/ovn-dbctl.c | 7 +++++++
  1 file changed, 7 insertions(+)

diff --git a/utilities/ovn-dbctl.c b/utilities/ovn-dbctl.c
index a292e589d..c4cc8c9b2 100644
--- a/utilities/ovn-dbctl.c
+++ b/utilities/ovn-dbctl.c
@@ -202,6 +202,13 @@ ovn_dbctl_main(int argc, char *argv[],
          error = ctl_parse_commands(argc - optind, argv_ + optind,
                                     &local_options, &commands, &n_commands);
          if (error) {
+            ovsdb_idl_destroy(idl);
+            idl = the_idl = NULL;
+
+            for (int i = 0; i < argc; i++) {
+                free(argv_[i]);
+            }
+            free(argv_);
              ctl_fatal("%s", error);
          }

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

Reply via email to