ctdb-recoverd: Fix a local memory leak

The memory is allocated off the memory context used by the current
iteration of main loop.  It is freed when main loop completes the fix
doesn't require backporting to stable branches.  However, it is sloppy
so it is worth fixing.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14466
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>

diff --git a/ctdb/server/ctdb_recoverd.c b/ctdb/server/ctdb_recoverd.c
index 41f029e6bca5ce7b10c46c608b29dac817853a96..5d6cfa2d0094056ad5543b43fac07e644d8235b9 100644 (file)
--- a/ctdb/server/ctdb_recoverd.c
+++ b/ctdb/server/ctdb_recoverd.c
@@ -2261,6 +2261,7 @@ static int get_remote_nodemaps(struct ctdb_context *ctdb,
                                        async_getnodemap_callback,
                                        NULL,
                                        remote_nodemaps);
+       talloc_free(nodes);
        return ret;
 }