Attachment 11255 Details for Bug 11398 – Patch for master

[patch] Patch for master

0001-ctdb-daemon-Return-correct-sequence-number-for-CONTR.patch (text/plain), 1.15 KB, created by Amitay Isaacs on 2015-07-14 07:48:31 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Amitay Isaacs

Created: 2015-07-14 07:48:31 UTC

Size: 1.15 KB

patch

obsolete

>From c7871ebf5fbd452131b436d0eecad8521c5f1ddf Mon Sep 17 00:00:00 2001
>From: Amitay Isaacs <amitay@gmail.com>
>Date: Tue, 14 Jul 2015 16:54:59 +1000
>Subject: [PATCH] ctdb-daemon: Return correct sequence number for
> CONTROL_GET_DB_SEQNUM
>
>Due to the missing cast of uint64_t, CONTROL_GET_DB_SEQNUM always returned
>seqnum <= 256.
>
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=11398
>
>Signed-off-by: Amitay Isaacs <amitay@gmail.com>
>Reviewed-by: Martin Schwenke <martin@meltin.net>
>Reviewed-by: Volker Lendecke <vl@samba.org>
>---
> ctdb/server/ctdb_persistent.c | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
>diff --git a/ctdb/server/ctdb_persistent.c b/ctdb/server/ctdb_persistent.c
>index e28622f..5c54b9e 100644
>--- a/ctdb/server/ctdb_persistent.c
>+++ b/ctdb/server/ctdb_persistent.c
>@@ -369,14 +369,11 @@ int32_t ctdb_control_get_db_seqnum(struct ctdb_context *ctdb,
> 	}
> 
> 	outdata->dsize = sizeof(uint64_t);
>-	outdata->dptr = (uint8_t *)talloc_zero(outdata, uint64_t);
>+	outdata->dptr = talloc_memdup(outdata, &seqnum, sizeof(uint64_t));
> 	if (outdata->dptr == NULL) {
> 		ret = -1;
>-		goto done;
> 	}
> 
>-	*(outdata->dptr) = seqnum;
>-
> done:
> 	return ret;
> }
>-- 
>2.4.3
>

Flags:

vl: review+

Actions: View

Attachments on bug 11398: 11254 | 11255 | 11256