s3:dbwrap_ctdb: rename db_ctdb_own_record() -> db_ctdb_can_use_local_copy()
authorMichael Adam <obnox@samba.org>
Fri, 29 Jun 2012 08:47:56 +0000 (10:47 +0200)
committerMichael Adam <obnox@samba.org>
Fri, 29 Jun 2012 13:21:19 +0000 (15:21 +0200)
This matches the function's intention much closer, since it
also tests whether we have a valid local read only copy.

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>

source3/lib/dbwrap/dbwrap_ctdb.c

index 922a5cf665d14350fe7e13261bce8d32e9c03c84..e5662e03575745a98eb9a6518086e5dc1d449809 100644 (file)
@@ -1004,7 +1004,7 @@ static int db_ctdb_record_destr(struct db_record* data)
 }
 
 /* Do I own this record? */
-static bool db_ctdb_own_record(TDB_DATA ctdb_data, bool read_only)
+static bool db_ctdb_can_use_local_copy(TDB_DATA ctdb_data, bool read_only)
 {
        struct ctdb_ltdb_header *hdr;
 
@@ -1098,7 +1098,7 @@ again:
         * take the shortcut and just return it.
         */
 
-       if (!db_ctdb_own_record(ctdb_data, false)) {
+       if (!db_ctdb_can_use_local_copy(ctdb_data, false)) {
                SAFE_FREE(ctdb_data.dptr);
                tdb_chainunlock(ctx->wtdb->tdb, key);
                talloc_set_destructor(result, NULL);
@@ -1222,7 +1222,7 @@ static NTSTATUS db_ctdb_fetch(struct db_context *db, TALLOC_CTX *mem_ctx,
         * take the shortcut and just return it.
         * we bypass the dmaster check for persistent databases
         */
-       if (db_ctdb_own_record(ctdb_data, true)) {
+       if (db_ctdb_can_use_local_copy(ctdb_data, true)) {
                /* we are the dmaster - avoid the ctdb protocol op */
 
                data->dsize = ctdb_data.dsize - sizeof(struct ctdb_ltdb_header);