Merge branch 'v4-0-test' of ssh://git.samba.org/data/git/samba into registry
[kai/samba.git] / source4 / wrepl_server / wrepl_scavenging.c
index 8f7c7b8e047327ed959726ae5dc73c35958542ad..a5cd36797d5193cf8a08a3ea69075b0ce2eed278 100644 (file)
@@ -7,7 +7,7 @@
    
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
-   the Free Software Foundation; either version 2 of the License, or
+   the Free Software Foundation; either version 3 of the License, or
    (at your option) any later version.
    
    This program is distributed in the hope that it will be useful,
@@ -16,8 +16,7 @@
    GNU General Public License for more details.
    
    You should have received a copy of the GNU General Public License
-   along with this program; if not, write to the Free Software
-   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
 #include "includes.h"
@@ -30,6 +29,7 @@
 #include "smbd/service_task.h"
 #include "lib/messaging/irpc.h"
 #include "librpc/gen_ndr/ndr_irpc.h"
+#include "librpc/gen_ndr/ndr_nbt.h"
 
 const char *wreplsrv_owner_filter(struct wreplsrv_service *service,
                                  TALLOC_CTX *mem_ctx,
@@ -56,11 +56,12 @@ static NTSTATUS wreplsrv_scavenging_owned_records(struct wreplsrv_service *servi
        time_t now = time(NULL);
        const char *now_timestr;
        const char *action;
-       const char *old_state;
+       const char *old_state=NULL;
+       const char *new_state=NULL;
        uint32_t modify_flags;
-       BOOL modify_record;
-       BOOL delete_record;
-       BOOL delete_tombstones;
+       bool modify_record;
+       bool delete_record;
+       bool delete_tombstones;
        struct timeval tombstone_extra_time;
 
        now_timestr = ldb_timestring(tmp_mem, now);
@@ -70,7 +71,7 @@ static NTSTATUS wreplsrv_scavenging_owned_records(struct wreplsrv_service *servi
        NT_STATUS_HAVE_NO_MEMORY(owner_filter);
        filter = talloc_asprintf(tmp_mem,
                                 "(&%s(objectClass=winsRecord)"
-                                "(expireTime<=%s)(!(isStatic=1)))",
+                                "(expireTime<=%s))",
                                 owner_filter, now_timestr);
        NT_STATUS_HAVE_NO_MEMORY(filter);
        ret = ldb_search(service->wins_db->ldb, NULL, LDB_SCOPE_SUBTREE, filter, NULL, &res);
@@ -84,47 +85,46 @@ static NTSTATUS wreplsrv_scavenging_owned_records(struct wreplsrv_service *servi
        delete_tombstones = timeval_expired(&tombstone_extra_time);
 
        for (i=0; i < res->count; i++) {
-               status = winsdb_record(service->wins_db, res->msgs[i], tmp_mem, &rec);
+               /*
+                * we pass '0' as 'now' here,
+                * because we want to get the raw timestamps which are in the DB
+                */
+               status = winsdb_record(service->wins_db, res->msgs[i], tmp_mem, 0, &rec);
                NT_STATUS_NOT_OK_RETURN(status);
                talloc_free(res->msgs[i]);
 
-               if (rec->is_static) {
-                       DEBUG(0,("%s: corrupted record: %s\n",
-                               __location__, nbt_name_string(rec, rec->name)));
-                       return NT_STATUS_INTERNAL_DB_CORRUPTION;
-               }
-
-               if (rec->expire_time > now) {
-                       DEBUG(0,("%s: corrupted record: %s\n",
-                               __location__, nbt_name_string(rec, rec->name)));
-                       return NT_STATUS_INTERNAL_DB_CORRUPTION;
-               }
-
                modify_flags    = 0;
-               modify_record   = False;
-               delete_record   = False;
+               modify_record   = false;
+               delete_record   = false;
 
                switch (rec->state) {
                case WREPL_STATE_ACTIVE:
                        old_state       = "active";
-                       rec->state      = WREPL_STATE_RELEASED;
-                       rec->expire_time= service->config.tombstone_interval + now;
+                       new_state       = "active";
+                       if (!rec->is_static) {
+                               new_state       = "released";
+                               rec->state      = WREPL_STATE_RELEASED;
+                               rec->expire_time= service->config.tombstone_interval + now;
+                       }
                        modify_flags    = 0;
-                       modify_record   = True;
+                       modify_record   = true;
                        break;
 
                case WREPL_STATE_RELEASED:
                        old_state       = "released";
+                       new_state       = "tombstone";
                        rec->state      = WREPL_STATE_TOMBSTONE;
                        rec->expire_time= service->config.tombstone_timeout + now;
                        modify_flags    = WINSDB_FLAG_ALLOC_VERSION | WINSDB_FLAG_TAKE_OWNERSHIP;
-                       modify_record   = True;
+                       modify_record   = true;
                        break;
 
                case WREPL_STATE_TOMBSTONE:
                        old_state       = "tombstone";
+                       new_state       = "tombstone";
                        if (!delete_tombstones) break;
-                       delete_record = True;
+                       new_state       = "deleted";
+                       delete_record = true;
                        break;
 
                case WREPL_STATE_RESERVED:
@@ -145,11 +145,11 @@ static NTSTATUS wreplsrv_scavenging_owned_records(struct wreplsrv_service *servi
                }
 
                if (ret != NBT_RCODE_OK) {
-                       DEBUG(1,("WINS scavenging: failed to %s name %s (owned:%s): error:%u\n",
-                               action, nbt_name_string(rec, rec->name), old_state, ret));
+                       DEBUG(1,("WINS scavenging: failed to %s name %s (owned:%s -> owned:%s): error:%u\n",
+                               action, nbt_name_string(rec, rec->name), old_state, new_state, ret));
                } else {
-                       DEBUG(4,("WINS scavenging: %s name: %s (owned:%s)\n",
-                               action, nbt_name_string(rec, rec->name), old_state));
+                       DEBUG(4,("WINS scavenging: %s name: %s (owned:%s -> owned:%s)\n",
+                               action, nbt_name_string(rec, rec->name), old_state, new_state));
                }
 
                talloc_free(rec);
@@ -170,11 +170,12 @@ static NTSTATUS wreplsrv_scavenging_replica_non_active_records(struct wreplsrv_s
        time_t now = time(NULL);
        const char *now_timestr;
        const char *action;
-       const char *old_state;
+       const char *old_state=NULL;
+       const char *new_state=NULL;
        uint32_t modify_flags;
-       BOOL modify_record;
-       BOOL delete_record;
-       BOOL delete_tombstones;
+       bool modify_record;
+       bool delete_record;
+       bool delete_tombstones;
        struct timeval tombstone_extra_time;
 
        now_timestr = ldb_timestring(tmp_mem, now);
@@ -184,7 +185,7 @@ static NTSTATUS wreplsrv_scavenging_replica_non_active_records(struct wreplsrv_s
        NT_STATUS_HAVE_NO_MEMORY(owner_filter);
        filter = talloc_asprintf(tmp_mem,
                                 "(&(!%s)(objectClass=winsRecord)"
-                                "(!(recordState=%u))(expireTime<=%s)(!(isStatic=1)))",
+                                "(!(recordState=%u))(expireTime<=%s))",
                                 owner_filter, WREPL_STATE_ACTIVE, now_timestr);
        NT_STATUS_HAVE_NO_MEMORY(filter);
        ret = ldb_search(service->wins_db->ldb, NULL, LDB_SCOPE_SUBTREE, filter, NULL, &res);
@@ -198,25 +199,17 @@ static NTSTATUS wreplsrv_scavenging_replica_non_active_records(struct wreplsrv_s
        delete_tombstones = timeval_expired(&tombstone_extra_time);
 
        for (i=0; i < res->count; i++) {
-               status = winsdb_record(service->wins_db, res->msgs[i], tmp_mem, &rec);
+               /*
+                * we pass '0' as 'now' here,
+                * because we want to get the raw timestamps which are in the DB
+                */
+               status = winsdb_record(service->wins_db, res->msgs[i], tmp_mem, 0, &rec);
                NT_STATUS_NOT_OK_RETURN(status);
                talloc_free(res->msgs[i]);
 
-               if (rec->is_static) {
-                       DEBUG(0,("%s: corrupted record: %s\n",
-                               __location__, nbt_name_string(rec, rec->name)));
-                       return NT_STATUS_INTERNAL_DB_CORRUPTION;
-               }
-
-               if (rec->expire_time > now) {
-                       DEBUG(0,("%s: corrupted record: %s\n",
-                               __location__, nbt_name_string(rec, rec->name)));
-                       return NT_STATUS_INTERNAL_DB_CORRUPTION;
-               }
-
                modify_flags    = 0;
-               modify_record   = False;
-               delete_record   = False;
+               modify_record   = false;
+               delete_record   = false;
 
                switch (rec->state) {
                case WREPL_STATE_ACTIVE:
@@ -226,16 +219,19 @@ static NTSTATUS wreplsrv_scavenging_replica_non_active_records(struct wreplsrv_s
 
                case WREPL_STATE_RELEASED:
                        old_state       = "released";
+                       new_state       = "tombstone";
                        rec->state      = WREPL_STATE_TOMBSTONE;
                        rec->expire_time= service->config.tombstone_timeout + now;
                        modify_flags    = 0;
-                       modify_record   = True;
+                       modify_record   = true;
                        break;
 
                case WREPL_STATE_TOMBSTONE:
                        old_state       = "tombstone";
+                       new_state       = "tombstone";
                        if (!delete_tombstones) break;
-                       delete_record = True;
+                       new_state       = "deleted";
+                       delete_record = true;
                        break;
 
                case WREPL_STATE_RESERVED:
@@ -256,11 +252,11 @@ static NTSTATUS wreplsrv_scavenging_replica_non_active_records(struct wreplsrv_s
                }
 
                if (ret != NBT_RCODE_OK) {
-                       DEBUG(1,("WINS scavenging: failed to %s name %s (replica:%s): error:%u\n",
-                               action, nbt_name_string(rec, rec->name), old_state, ret));
+                       DEBUG(1,("WINS scavenging: failed to %s name %s (replica:%s -> replica:%s): error:%u\n",
+                               action, nbt_name_string(rec, rec->name), old_state, new_state, ret));
                } else {
-                       DEBUG(4,("WINS scavenging: %s name: %s (replica:%s)\n",
-                               action, nbt_name_string(rec, rec->name), old_state));
+                       DEBUG(4,("WINS scavenging: %s name: %s (replica:%s -> replica:%s)\n",
+                               action, nbt_name_string(rec, rec->name), old_state, new_state));
                }
 
                talloc_free(rec);
@@ -283,9 +279,12 @@ static void verify_handler(struct irpc_request *ireq)
        struct winsdb_record *rec = s->rec;
        const char *action;
        const char *old_state = "active";
-       BOOL modify_record = False;
-       BOOL delete_record = False;
-       BOOL different = False;
+       const char *new_state = "active";
+       const char *new_owner = "replica";
+       uint32_t modify_flags = 0;
+       bool modify_record = false;
+       bool delete_record = false;
+       bool different = false;
        int ret;
        NTSTATUS status;
        uint32_t i, j;
@@ -299,42 +298,62 @@ static void verify_handler(struct irpc_request *ireq)
         */
        status = irpc_call_recv(ireq);
        if (NT_STATUS_EQUAL(NT_STATUS_OBJECT_NAME_NOT_FOUND, status)) {
-               delete_record = True;
+               delete_record = true;
+               new_state = "deleted";
        } else if (NT_STATUS_IS_OK(status) && rec->type != WREPL_TYPE_GROUP) {
                for (i=0; i < s->r.out.num_addrs; i++) {
-                       BOOL found = False;
+                       bool found = false;
                        for (j=0; rec->addresses[j]; j++) {
                                if (strcmp(s->r.out.addrs[i].addr, rec->addresses[j]->address) == 0) {
-                                       found = True;
+                                       found = true;
                                        break;
                                }
                        }
                        if (!found) {
-                               different = True;
+                               different = true;
                                break;
                        }
                }
        } else if (NT_STATUS_IS_OK(status) && rec->type == WREPL_TYPE_GROUP) {
-               if (s->r.out.num_addrs != 1 || strcmp(s->r.out.addrs[i].addr, "255.255.255.255") != 0) {
-                       different = True;
+               if (s->r.out.num_addrs != 1 || strcmp(s->r.out.addrs[0].addr, "255.255.255.255") != 0) {
+                       different = true;
                }
        }
 
        if (different) {
-               DEBUG(0,("WINS scavenging: replica %s verify got different addresses from winsserver: %s: deleting record\n",
+               /*
+                * if the reply from the owning wins server has different addresses
+                * then take the ownership of the record and make it a tombstone
+                * this will then hopefully replicated to the original owner of the record
+                * which will then propagate it's own record, so that the current record will
+                * be replicated to to us
+                */
+               DEBUG(0,("WINS scavenging: replica %s verify got different addresses from winsserver: %s: tombstoning record\n",
                        nbt_name_string(rec, rec->name), rec->wins_owner));
-               delete_record = True;
+
+               rec->state      = WREPL_STATE_TOMBSTONE;
+               rec->expire_time= time(NULL) + s->service->config.tombstone_timeout;
+               for (i=0; rec->addresses[i]; i++) {
+                       rec->addresses[i]->expire_time = rec->expire_time;
+               }
+               modify_record   = true;
+               modify_flags    = WINSDB_FLAG_ALLOC_VERSION | WINSDB_FLAG_TAKE_OWNERSHIP;
+               new_state       = "tombstone";
+               new_owner       = "owned";
        } else if (NT_STATUS_IS_OK(status)) {
+               /* if the addresses are the same, just update the timestamps */
                rec->expire_time = time(NULL) + s->service->config.verify_interval;
                for (i=0; rec->addresses[i]; i++) {
                        rec->addresses[i]->expire_time = rec->expire_time;
                }
-               modify_record = True;
+               modify_record   = true;
+               modify_flags    = 0;
+               new_state       = "active";
        }
 
        if (modify_record) {
                action = "modify";
-               ret = winsdb_modify(s->service->wins_db, rec, 0);
+               ret = winsdb_modify(s->service->wins_db, rec, modify_flags);
        } else if (delete_record) {
                action = "delete";
                ret = winsdb_delete(s->service->wins_db, rec);
@@ -344,11 +363,12 @@ static void verify_handler(struct irpc_request *ireq)
        }
 
        if (ret != NBT_RCODE_OK) {
-               DEBUG(1,("WINS scavenging: failed to %s name %s (replica:%s): error:%u\n",
-                       action, nbt_name_string(rec, rec->name), old_state, ret));
+               DEBUG(1,("WINS scavenging: failed to %s name %s (replica:%s -> %s:%s): error:%u\n",
+                       action, nbt_name_string(rec, rec->name), old_state, new_owner, new_state, ret));
        } else {
-               DEBUG(4,("WINS scavenging: %s name: %s (replica:%s): %s: %s\n",
-                       action, nbt_name_string(rec, rec->name), old_state, rec->wins_owner, nt_errstr(status)));
+               DEBUG(4,("WINS scavenging: %s name: %s (replica:%s -> %s:%s): %s: %s\n",
+                       action, nbt_name_string(rec, rec->name), old_state, new_owner, new_state,
+                       rec->wins_owner, nt_errstr(status)));
        }
 
        talloc_free(s);
@@ -367,10 +387,10 @@ static NTSTATUS wreplsrv_scavenging_replica_active_records(struct wreplsrv_servi
        const char *now_timestr;
        struct irpc_request *ireq;
        struct verify_state *s;
-       uint32_t *nbt_servers;
+       struct server_id *nbt_servers;
 
-       nbt_servers = irpc_servers_byname(service->task->msg_ctx, "nbt_server");
-       if ((nbt_servers == NULL) || (nbt_servers[0] == 0)) {
+       nbt_servers = irpc_servers_byname(service->task->msg_ctx, tmp_mem, "nbt_server");
+       if ((nbt_servers == NULL) || (nbt_servers[0].id == 0)) {
                return NT_STATUS_INTERNAL_ERROR;
        }
 
@@ -381,7 +401,7 @@ static NTSTATUS wreplsrv_scavenging_replica_active_records(struct wreplsrv_servi
        NT_STATUS_HAVE_NO_MEMORY(owner_filter);
        filter = talloc_asprintf(tmp_mem,
                                 "(&(!%s)(objectClass=winsRecord)"
-                                "(recordState=%u)(expireTime<=%s)(!(isStatic=1)))",
+                                "(recordState=%u)(expireTime<=%s))",
                                 owner_filter, WREPL_STATE_ACTIVE, now_timestr);
        NT_STATUS_HAVE_NO_MEMORY(filter);
        ret = ldb_search(service->wins_db->ldb, NULL, LDB_SCOPE_SUBTREE, filter, NULL, &res);
@@ -390,22 +410,14 @@ static NTSTATUS wreplsrv_scavenging_replica_active_records(struct wreplsrv_servi
        DEBUG(10,("WINS scavenging: filter '%s' count %d\n", filter, res->count));
 
        for (i=0; i < res->count; i++) {
-               status = winsdb_record(service->wins_db, res->msgs[i], tmp_mem, &rec);
+               /*
+                * we pass '0' as 'now' here,
+                * because we want to get the raw timestamps which are in the DB
+                */
+               status = winsdb_record(service->wins_db, res->msgs[i], tmp_mem, 0, &rec);
                NT_STATUS_NOT_OK_RETURN(status);
                talloc_free(res->msgs[i]);
 
-               if (rec->is_static) {
-                       DEBUG(0,("%s: corrupted record: %s\n",
-                               __location__, nbt_name_string(rec, rec->name)));
-                       return NT_STATUS_INTERNAL_DB_CORRUPTION;
-               }
-
-               if (rec->expire_time > now) {
-                       DEBUG(0,("%s: corrupted record: %s\n",
-                               __location__, nbt_name_string(rec, rec->name)));
-                       return NT_STATUS_INTERNAL_DB_CORRUPTION;
-               }
-
                if (rec->state != WREPL_STATE_ACTIVE) {
                        DEBUG(0,("%s: corrupted record: %s\n",
                                __location__, nbt_name_string(rec, rec->name)));
@@ -422,7 +434,8 @@ static NTSTATUS wreplsrv_scavenging_replica_active_records(struct wreplsrv_servi
                 *             but we use a wins name query
                 */
                DEBUG(0,("ask wins server '%s' if '%s' with version_id:%llu still exists\n",
-                       rec->wins_owner, nbt_name_string(rec, rec->name), rec->version));
+                        rec->wins_owner, nbt_name_string(rec, rec->name), 
+                        (unsigned long long)rec->version));
 
                s = talloc_zero(tmp_mem, struct verify_state);
                NT_STATUS_HAVE_NO_MEMORY(s);
@@ -455,14 +468,14 @@ NTSTATUS wreplsrv_scavenging_run(struct wreplsrv_service *service)
 {
        NTSTATUS status;
        TALLOC_CTX *tmp_mem;
-       BOOL skip_first_run = False;
+       bool skip_first_run = false;
 
        if (!timeval_expired(&service->scavenging.next_run)) {
                return NT_STATUS_OK;
        }
 
        if (timeval_is_zero(&service->scavenging.next_run)) {
-               skip_first_run = True;
+               skip_first_run = true;
        }
 
        service->scavenging.next_run = timeval_current_ofs(service->config.scavenging_interval, 0);
@@ -484,23 +497,26 @@ NTSTATUS wreplsrv_scavenging_run(struct wreplsrv_service *service)
        DEBUG(4,("wreplsrv_scavenging_run(): start\n"));
 
        tmp_mem = talloc_new(service);
-       service->scavenging.processing = True;
+       NT_STATUS_HAVE_NO_MEMORY(tmp_mem);
+       service->scavenging.processing = true;
        status = wreplsrv_scavenging_owned_records(service,tmp_mem);
-       service->scavenging.processing = False;
+       service->scavenging.processing = false;
        talloc_free(tmp_mem);
        NT_STATUS_NOT_OK_RETURN(status);
 
        tmp_mem = talloc_new(service);  
-       service->scavenging.processing = True;
+       NT_STATUS_HAVE_NO_MEMORY(tmp_mem);
+       service->scavenging.processing = true;
        status = wreplsrv_scavenging_replica_non_active_records(service, tmp_mem);
-       service->scavenging.processing = False;
+       service->scavenging.processing = false;
        talloc_free(tmp_mem);
        NT_STATUS_NOT_OK_RETURN(status);
 
        tmp_mem = talloc_new(service);
-       service->scavenging.processing = True;
+       NT_STATUS_HAVE_NO_MEMORY(tmp_mem);
+       service->scavenging.processing = true;
        status = wreplsrv_scavenging_replica_active_records(service, tmp_mem);
-       service->scavenging.processing = False;
+       service->scavenging.processing = false;
        talloc_free(tmp_mem);
        NT_STATUS_NOT_OK_RETURN(status);