s3-dbwrap: Rewrite dbwrap_fallback_parse_record based on dbwrap_fetch_locked
[kai/samba.git] / source3 / lib / dbwrap / dbwrap.c
index 75d38b5920d76bc3f9ff12bff7557aaef8ba1dfd..0635d76b2cba189327c1690c305a27b3d0b9b9a4 100644 (file)
@@ -1,20 +1,20 @@
-/* 
+/*
    Unix SMB/CIFS implementation.
    Database interface wrapper
    Copyright (C) Jim McDonough <jmcd@us.ibm.com> 2006
 
    Major code contributions from Aleksey Fedoseev (fedoseev@ru.ibm.com)
-   
+
    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 3 of the License, or
    (at your option) any later version.
-   
+
    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
    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, see <http://www.gnu.org/licenses/>.
 */
  * Fall back using fetch_locked if no genuine fetch operation is provided
  */
 
-int dbwrap_fallback_fetch(struct db_context *db, TALLOC_CTX *mem_ctx,
-                         TDB_DATA key, TDB_DATA *data)
+static NTSTATUS dbwrap_fallback_fetch(struct db_context *db,
+                                     TALLOC_CTX *mem_ctx,
+                                     TDB_DATA key, TDB_DATA *data)
 {
        struct db_record *rec;
 
-       if (!(rec = db->fetch_locked(db, mem_ctx, key))) {
-               return -1;
+       rec = dbwrap_fetch_locked(db, mem_ctx, key);
+       if (rec == NULL) {
+               return NT_STATUS_UNSUCCESSFUL;
        }
 
        data->dsize = rec->value.dsize;
        data->dptr = talloc_move(mem_ctx, &rec->value.dptr);
        TALLOC_FREE(rec);
-       return 0;
+       return NT_STATUS_OK;
 }
 
 /*
@@ -49,47 +51,55 @@ int dbwrap_fallback_fetch(struct db_context *db, TALLOC_CTX *mem_ctx,
 
 static int dbwrap_fallback_exists(struct db_context *db, TDB_DATA key)
 {
-       int res = dbwrap_parse_record(db, key, NULL, NULL);
-       return  ( res == -1) ? 0 : 1;
+       NTSTATUS status = dbwrap_parse_record(db, key, NULL, NULL);
+       return NT_STATUS_IS_OK(status) ? 1 : 0;
 }
 
 /*
  * Fall back using fetch if no genuine parse operation is provided
  */
 
-int dbwrap_fallback_parse_record(struct db_context *db, TDB_DATA key,
-                                int (*parser)(TDB_DATA key,
-                                              TDB_DATA data,
-                                              void *private_data),
-                                void *private_data)
+static NTSTATUS dbwrap_fallback_parse_record(struct db_context *db, TDB_DATA key,
+                                            void (*parser)(TDB_DATA key,
+                                                           TDB_DATA data,
+                                                           void *private_data),
+                                            void *private_data)
 {
+       struct db_record *rec;
        TDB_DATA data;
-       int res;
 
-       res = db->fetch(db, talloc_tos(), key, &data);
-       if (res != 0) {
-               return -1;
+       rec = dbwrap_fetch_locked(db, talloc_tos(), key);
+       if (rec == NULL) {
+               return NT_STATUS_NOT_FOUND;
+       }
+       data = dbwrap_record_get_value(rec);
+
+       data.dptr = talloc_memdup(talloc_tos(), data.dptr, data.dsize);
+       TALLOC_FREE(rec);
+       if (data.dptr == NULL) {
+               return NT_STATUS_NO_MEMORY;
        }
 
-       res = parser(key, data, private_data);
+       parser(key, data, private_data);
+
        TALLOC_FREE(data.dptr);
-       return res;
+       return NT_STATUS_OK;
 }
 
 
 static int delete_record(struct db_record *rec, void *data)
 {
-       NTSTATUS status = rec->delete_rec(rec);
+       NTSTATUS status = dbwrap_record_delete(rec);
        return NT_STATUS_IS_OK(status) ? 0 : -1;
 }
 
 /*
- * Fallback wipe ipmlementation using traverse and delete if no genuine
+ * Fallback wipe implementation using traverse and delete if no genuine
  * wipe operation is provided
  */
-int dbwrap_fallback_wipe(struct db_context *db)
+static int dbwrap_fallback_wipe(struct db_context *db)
 {
-       NTSTATUS status = dbwrap_trans_traverse(db, &delete_record, NULL);
+       NTSTATUS status = dbwrap_trans_traverse(db, delete_record, NULL);
        return NT_STATUS_IS_OK(status) ? 0 : -1;
 }
 
@@ -108,21 +118,33 @@ TDB_DATA dbwrap_record_get_value(const struct db_record *rec)
        return rec->value;
 }
 
-void *dbwrap_record_get_private_data(const struct db_record *rec)
+NTSTATUS dbwrap_record_store(struct db_record *rec, TDB_DATA data, int flags)
 {
-       return rec->private_data;
+       return rec->store(rec, data, flags);
 }
 
-TDB_DATA dbwrap_fetch(struct db_context *db, TALLOC_CTX *mem_ctx,
-                     TDB_DATA key)
+NTSTATUS dbwrap_record_delete(struct db_record *rec)
 {
-       TDB_DATA result;
+       return rec->delete_rec(rec);
+}
 
-       if (db->fetch(db, mem_ctx, key, &result) != 0) {
-               return make_tdb_data(NULL, 0);
-       }
+struct db_record *dbwrap_fetch_locked(struct db_context *db,
+                                     TALLOC_CTX *mem_ctx,
+                                     TDB_DATA key)
+{
+       return db->fetch_locked(db, mem_ctx, key);
+}
 
-       return result;
+NTSTATUS dbwrap_fetch(struct db_context *db, TALLOC_CTX *mem_ctx,
+                     TDB_DATA key, TDB_DATA *value)
+{
+       if (value == NULL) {
+               return NT_STATUS_INVALID_PARAMETER;
+       }
+       if (db->fetch == NULL) {
+               return dbwrap_fallback_fetch(db, mem_ctx, key, value);
+       }
+       return db->fetch(db, mem_ctx, key, value);
 }
 
 bool dbwrap_exists(struct db_context *db, TDB_DATA key)
@@ -142,12 +164,12 @@ NTSTATUS dbwrap_store(struct db_context *db, TDB_DATA key,
        struct db_record *rec;
        NTSTATUS status;
 
-       rec = db->fetch_locked(db, talloc_tos(), key);
+       rec = dbwrap_fetch_locked(db, talloc_tos(), key);
        if (rec == NULL) {
                return NT_STATUS_NO_MEMORY;
        }
 
-       status = rec->store(rec, data, flags);
+       status = dbwrap_record_store(rec, data, flags);
        TALLOC_FREE(rec);
        return status;
 }
@@ -157,11 +179,11 @@ NTSTATUS dbwrap_delete(struct db_context *db, TDB_DATA key)
        struct db_record *rec;
        NTSTATUS status;
 
-       rec = db->fetch_locked(db, talloc_tos(), key);
+       rec = dbwrap_fetch_locked(db, talloc_tos(), key);
        if (rec == NULL) {
                return NT_STATUS_NO_MEMORY;
        }
-       status = rec->delete_rec(rec);
+       status = dbwrap_record_delete(rec);
        TALLOC_FREE(rec);
        return status;
 }
@@ -202,29 +224,32 @@ NTSTATUS dbwrap_traverse_read(struct db_context *db,
        return NT_STATUS_OK;
 }
 
-static int dbwrap_null_parser(TDB_DATA key, TDB_DATA val, void* data)
+static void dbwrap_null_parser(TDB_DATA key, TDB_DATA val, void* data)
 {
-       return 0;
+       return;
 }
 
-int dbwrap_parse_record(struct db_context *db, TDB_DATA key,
-                       int (*parser)(TDB_DATA key, TDB_DATA data,
-                                     void *private_data),
-                       void *private_data)
+NTSTATUS dbwrap_parse_record(struct db_context *db, TDB_DATA key,
+                            void (*parser)(TDB_DATA key, TDB_DATA data,
+                                           void *private_data),
+                            void *private_data)
 {
        if (parser == NULL) {
                parser = dbwrap_null_parser;
        }
 
-       if (db->parse_record) {
-               return db->parse_record(db, key, parser, private_data);
-       } else {
-               return dbwrap_fallback_parse_record(db, key, parser, private_data);
+       if (db->parse_record == NULL) {
+               return dbwrap_fallback_parse_record(db, key, parser,
+                                                   private_data);
        }
+       return db->parse_record(db, key, parser, private_data);
 }
 
 int dbwrap_wipe(struct db_context *db)
 {
+       if (db->wipe == NULL) {
+               return dbwrap_fallback_wipe(db);
+       }
        return db->wipe(db);
 }