From 08e229df43ed62991ea966e42e966afcacb4cb55 Mon Sep 17 00:00:00 2001 From: Martin Schwenke Date: Thu, 11 Apr 2019 16:56:32 +1000 Subject: [PATCH] ctdb-tools: Fix ctdb dumpmemory to avoid printing trailing NUL Fix ctdb rddumpmemory too. BUG: https://bugzilla.samba.org/show_bug.cgi?id=13923 Signed-off-by: Martin Schwenke Reviewed-by: Amitay Isaacs (cherry picked from commit f78d9388fb459dc83fafb4da6e683e3137ad40e1) --- ctdb/tools/ctdb.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/ctdb/tools/ctdb.c b/ctdb/tools/ctdb.c index 352881a62fe..71a14d67cb0 100644 --- a/ctdb/tools/ctdb.c +++ b/ctdb/tools/ctdb.c @@ -2434,8 +2434,8 @@ static int control_dumpmemory(TALLOC_CTX *mem_ctx, struct ctdb_context *ctdb, return ret; } - n = write(1, mem_str, strlen(mem_str)+1); - if (n < 0 || n != strlen(mem_str)+1) { + n = write(1, mem_str, strlen(mem_str)); + if (n < 0 || n != strlen(mem_str)) { fprintf(stderr, "Failed to write talloc summary\n"); return 1; } @@ -2446,10 +2446,12 @@ static int control_dumpmemory(TALLOC_CTX *mem_ctx, struct ctdb_context *ctdb, static void dump_memory(uint64_t srvid, TDB_DATA data, void *private_data) { bool *done = (bool *)private_data; + size_t len; ssize_t n; - n = write(1, data.dptr, data.dsize); - if (n < 0 || n != data.dsize) { + len = strnlen((const char *)data.dptr, data.dsize); + n = write(1, data.dptr, len); + if (n < 0 || n != len) { fprintf(stderr, "Failed to write talloc summary\n"); }