s3:lib/gencache: fix memory leak in error path
[metze/samba/wip.git] / source3 / lib / gencache.c
index 89badcd4f97dddef8994c937b155f459c39d5de0..7bab9256483e208da0732d45339b7870c35f27b5 100644 (file)
@@ -5,20 +5,20 @@
    by various parts of the Samba code
 
    Copyright (C) Rafal Szczesniak    2002
-   
+   Copyright (C) Volker Lendecke     2009
+
    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,
    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, 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"
 #define DBGC_CLASS DBGC_TDB
 
 #define TIMEOUT_LEN 12
-#define CACHE_DATA_FMT "%12u/%s"
+#define CACHE_DATA_FMT "%12u/"
+#define READ_CACHE_DATA_FMT_TEMPLATE "%%12u/%%%us"
+#define BLOB_TYPE "DATA_BLOB"
+#define BLOB_TYPE_LEN 9
 
-static TDB_CONTEXT *cache;
+static struct tdb_context *cache;
+static struct tdb_context *cache_notrans;
 
 /**
  * @file gencache.c
@@ -47,180 +51,531 @@ static TDB_CONTEXT *cache;
  *         false on failure
  **/
 
-BOOL gencache_init(void)
+static bool gencache_init(void)
 {
        char* cache_fname = NULL;
-       
+       int open_flags = O_RDWR|O_CREAT;
+       bool first_try = true;
+
        /* skip file open if it's already opened */
        if (cache) return True;
 
-       asprintf(&cache_fname, "%s/%s", lp_lockdir(), "gencache.tdb");
-       if (cache_fname)
-               DEBUG(5, ("Opening cache file at %s\n", cache_fname));
-       else {
-               DEBUG(0, ("Filename allocation failed.\n"));
-               return False;
+       cache_fname = lock_path("gencache.tdb");
+
+       DEBUG(5, ("Opening cache file at %s\n", cache_fname));
+
+again:
+       cache = tdb_open_log(cache_fname, 0, TDB_DEFAULT|TDB_INCOMPATIBLE_HASH, open_flags, 0644);
+       if (cache) {
+               int ret;
+               ret = tdb_check(cache, NULL, NULL);
+               if (ret != 0) {
+                       tdb_close(cache);
+                       cache = NULL;
+                       if (!first_try) {
+                               DEBUG(0, ("gencache_init: tdb_check(%s) failed\n",
+                                         cache_fname));
+                               return false;
+                       }
+                       first_try = false;
+                       DEBUG(0, ("gencache_init: tdb_check(%s) failed - retry after CLEAR_IF_FIRST\n",
+                                 cache_fname));
+                       cache = tdb_open_log(cache_fname, 0, TDB_CLEAR_IF_FIRST|TDB_INCOMPATIBLE_HASH, open_flags, 0644);
+                       if (cache) {
+                               tdb_close(cache);
+                               cache = NULL;
+                               goto again;
+                       }
+               }
        }
 
-       cache = tdb_open_log(cache_fname, 0, TDB_DEFAULT,
-                            O_RDWR|O_CREAT, 0644);
+       if (!cache && (errno == EACCES)) {
+               open_flags = O_RDONLY;
+               cache = tdb_open_log(cache_fname, 0, TDB_DEFAULT|TDB_INCOMPATIBLE_HASH, open_flags,
+                                    0644);
+               if (cache) {
+                       DEBUG(5, ("gencache_init: Opening cache file %s read-only.\n", cache_fname));
+               }
+       }
 
-       SAFE_FREE(cache_fname);
        if (!cache) {
                DEBUG(5, ("Attempt to open gencache.tdb has failed.\n"));
                return False;
        }
+
+       cache_fname = lock_path("gencache_notrans.tdb");
+
+       DEBUG(5, ("Opening cache file at %s\n", cache_fname));
+
+       cache_notrans = tdb_open_log(cache_fname, 0, TDB_CLEAR_IF_FIRST|TDB_INCOMPATIBLE_HASH,
+                                    open_flags, 0644);
+       if (cache_notrans == NULL) {
+               DEBUG(5, ("Opening %s failed: %s\n", cache_fname,
+                         strerror(errno)));
+               tdb_close(cache);
+               cache = NULL;
+               return false;
+       }
+
        return True;
 }
 
-
-/**
- * Cache shutdown function. Closes opened cache tdb file.
- *
- * @return true on successful closing the cache or
- *         false on failure during cache shutdown
- **/
-BOOL gencache_shutdown(void)
+static TDB_DATA last_stabilize_key(void)
 {
-       int ret;
-       /* tdb_close routine returns -1 on error */
-       if (!cache) return False;
-       DEBUG(5, ("Closing cache file\n"));
-       ret = tdb_close(cache);
-       cache = NULL;
-       return ret != -1;
+       TDB_DATA result;
+       result.dptr = (uint8_t *)"@LAST_STABILIZED";
+       result.dsize = 17;
+       return result;
 }
 
-
 /**
  * Set an entry in the cache file. If there's no such
  * one, then add it.
  *
  * @param keystr string that represents a key of this entry
- * @param value text representation value being cached
+ * @param blob DATA_BLOB value being cached
  * @param timeout time when the value is expired
  *
  * @retval true when entry is successfuly stored
  * @retval false on failure
  **/
-BOOL gencache_set(const char *keystr, const char *value, time_t timeout)
+
+bool gencache_set_data_blob(const char *keystr, const DATA_BLOB *blob,
+                           time_t timeout)
 {
        int ret;
-       TDB_DATA keybuf, databuf;
-       char* valstr = NULL;
-       
-       /* fail completely if get null pointers passed */
-       SMB_ASSERT(keystr && value);
+       TDB_DATA databuf;
+       char* val;
+       time_t last_stabilize;
+       static int writecount;
+
+       if (tdb_data_cmp(string_term_tdb_data(keystr),
+                        last_stabilize_key()) == 0) {
+               DEBUG(10, ("Can't store %s as a key\n", keystr));
+               return false;
+       }
+
+       if ((keystr == NULL) || (blob == NULL)) {
+               return false;
+       }
 
        if (!gencache_init()) return False;
-       
-       asprintf(&valstr, CACHE_DATA_FMT, (int)timeout, value);
-       if (!valstr)
+
+       val = talloc_asprintf(talloc_tos(), CACHE_DATA_FMT, (int)timeout);
+       if (val == NULL) {
                return False;
+       }
+       val = talloc_realloc(NULL, val, char, talloc_array_length(val)-1);
+       if (val == NULL) {
+               return false;
+       }
+       val = (char *)talloc_append_blob(NULL, val, *blob);
+       if (val == NULL) {
+               return false;
+       }
 
-       keybuf.dptr = SMB_STRDUP(keystr);
-       keybuf.dsize = strlen(keystr)+1;
-       databuf.dptr = SMB_STRDUP(valstr);
-       databuf.dsize = strlen(valstr)+1;
-       DEBUG(10, ("Adding cache entry with key = %s; value = %s and timeout ="
-                  " %s (%d seconds %s)\n", keybuf.dptr, value,ctime(&timeout),
+       DEBUG(10, ("Adding cache entry with key = %s and timeout ="
+                  " %s (%d seconds %s)\n", keystr, ctime(&timeout),
                   (int)(timeout - time(NULL)), 
                   timeout > time(NULL) ? "ahead" : "in the past"));
 
-       ret = tdb_store(cache, keybuf, databuf, 0);
-       SAFE_FREE(valstr);
-       SAFE_FREE(keybuf.dptr);
-       SAFE_FREE(databuf.dptr);
-       
+       ret = tdb_store_bystring(
+               cache_notrans, keystr,
+               make_tdb_data((uint8_t *)val, talloc_array_length(val)),
+               0);
+       TALLOC_FREE(val);
+
+       if (ret != 0) {
+               return false;
+       }
+
+       /*
+        * Every 100 writes within a single process, stabilize the cache with
+        * a transaction. This is done to prevent a single transaction to
+        * become huge and chew lots of memory.
+        */
+       writecount += 1;
+       if (writecount > lp_parm_int(-1, "gencache", "stabilize_count", 100)) {
+               gencache_stabilize();
+               writecount = 0;
+               goto done;
+       }
+
+       /*
+        * Every 5 minutes, call gencache_stabilize() to not let grow
+        * gencache_notrans.tdb too large.
+        */
+
+       last_stabilize = 0;
+       databuf = tdb_fetch(cache_notrans, last_stabilize_key());
+       if ((databuf.dptr != NULL)
+           && (databuf.dptr[databuf.dsize-1] == '\0')) {
+               last_stabilize = atoi((char *)databuf.dptr);
+               SAFE_FREE(databuf.dptr);
+       }
+       if ((last_stabilize
+            + lp_parm_int(-1, "gencache", "stabilize_interval", 300))
+           < time(NULL)) {
+               gencache_stabilize();
+       }
+
+done:
        return ret == 0;
 }
 
-
 /**
- * Set existing entry to the cache file.
+ * Delete one entry from the cache file.
  *
  * @param keystr string that represents a key of this entry
- * @param valstr text representation value being cached
- * @param timeout time when the value is expired
  *
- * @retval true when entry is successfuly set
- * @retval false on failure
+ * @retval true upon successful deletion
+ * @retval false in case of failure
  **/
 
-BOOL gencache_set_only(const char *keystr, const char *valstr, time_t timeout)
+bool gencache_del(const char *keystr)
 {
-       int ret = -1;
-       TDB_DATA keybuf, databuf;
-       char *old_valstr, *datastr;
-       time_t old_timeout;
-       
-       /* fail completely if get null pointers passed */
-       SMB_ASSERT(keystr && valstr);
+       bool exists, was_expired;
+       bool ret = false;
+       DATA_BLOB value;
 
-       if (!gencache_init()) return False;
-                       
-       /* 
-        * Check whether entry exists in the cache
-        * Don't verify gencache_get exit code, since the entry may be expired
-        */     
-       gencache_get(keystr, &old_valstr, &old_timeout);
-       
-       if (!(old_valstr && old_timeout)) return False;
-               
-       DEBUG(10, ("Setting cache entry with key = %s; old value = %s and old timeout \
-                  = %s\n", keystr, old_valstr, ctime(&old_timeout)));
-
-       asprintf(&datastr, CACHE_DATA_FMT, (int)timeout, valstr);
-       keybuf.dptr = SMB_STRDUP(keystr);
-       keybuf.dsize = strlen(keystr)+1;
-       databuf.dptr = SMB_STRDUP(datastr);
-       databuf.dsize = strlen(datastr)+1;
-       DEBUGADD(10, ("New value = %s, new timeout = %s (%d seconds %s)", valstr,
-                     ctime(&timeout), (int)(timeout - time(NULL)),
-                     timeout > time(NULL) ? "ahead" : "in the past"));
-
-               
-       ret = tdb_store(cache, keybuf, databuf, TDB_REPLACE);
-
-       SAFE_FREE(datastr);
-       SAFE_FREE(old_valstr);
-       SAFE_FREE(keybuf.dptr);
-       SAFE_FREE(databuf.dptr);
-       
-       return ret == 0;
+       if (keystr == NULL) {
+               return false;
+       }
+
+       if (!gencache_init()) return False;     
+
+       DEBUG(10, ("Deleting cache entry (key = %s)\n", keystr));
+
+       /*
+        * We delete an element by setting its timeout to 0. This way we don't
+        * have to do a transaction on gencache.tdb every time we delete an
+        * element.
+        */
+
+       exists = gencache_get_data_blob(keystr, &value, NULL, &was_expired);
+
+       if (!exists && was_expired) {
+               /*
+                * gencache_get_data_blob has implicitly deleted this
+                * entry, so we have to return success here.
+                */
+               return true;
+       }
+
+       if (exists) {
+               data_blob_free(&value);
+               ret = gencache_set(keystr, "", 0);
+       }
+       return ret;
+}
+
+static bool gencache_pull_timeout(char *val, time_t *pres, char **pendptr)
+{
+       time_t res;
+       char *endptr;
+
+       if (val == NULL) {
+               return false;
+       }
+
+       res = strtol(val, &endptr, 10);
+
+       if ((endptr == NULL) || (*endptr != '/')) {
+               DEBUG(2, ("Invalid gencache data format: %s\n", val));
+               return false;
+       }
+       if (pres != NULL) {
+               *pres = res;
+       }
+       if (pendptr != NULL) {
+               *pendptr = endptr;
+       }
+       return true;
+}
+
+struct gencache_parse_state {
+       void (*parser)(time_t timeout, DATA_BLOB blob, void *private_data);
+       void *private_data;
+};
+
+static int gencache_parse_fn(TDB_DATA key, TDB_DATA data, void *private_data)
+{
+       struct gencache_parse_state *state;
+       DATA_BLOB blob;
+       time_t t;
+       char *endptr;
+       bool ret;
+
+       if (data.dptr == NULL) {
+               return -1;
+       }
+       ret = gencache_pull_timeout((char *)data.dptr, &t, &endptr);
+       if (!ret) {
+               return -1;
+       }
+       state = (struct gencache_parse_state *)private_data;
+       blob = data_blob_const(
+               endptr+1, data.dsize - PTR_DIFF(endptr+1, data.dptr));
+       state->parser(t, blob, state->private_data);
+       return 0;
+}
+
+bool gencache_parse(const char *keystr,
+                   void (*parser)(time_t timeout, DATA_BLOB blob,
+                                  void *private_data),
+                   void *private_data)
+{
+       struct gencache_parse_state state;
+       TDB_DATA key;
+       int ret;
+
+       if (keystr == NULL) {
+               return false;
+       }
+       if (tdb_data_cmp(string_term_tdb_data(keystr),
+                        last_stabilize_key()) == 0) {
+               return false;
+       }
+       if (!gencache_init()) {
+               return false;
+       }
+
+       key = string_term_tdb_data(keystr);
+       state.parser = parser;
+       state.private_data = private_data;
+
+       ret = tdb_parse_record(cache_notrans, key, gencache_parse_fn, &state);
+       if (ret != -1) {
+               return true;
+       }
+       ret = tdb_parse_record(cache, key, gencache_parse_fn, &state);
+       return (ret != -1);
+}
+
+struct gencache_get_data_blob_state {
+       DATA_BLOB *blob;
+       time_t timeout;
+       bool result;
+};
+
+static void gencache_get_data_blob_parser(time_t timeout, DATA_BLOB blob,
+                                         void *private_data)
+{
+       struct gencache_get_data_blob_state *state =
+               (struct gencache_get_data_blob_state *)private_data;
+
+       if (timeout == 0) {
+               state->result = false;
+               return;
+       }
+       state->timeout = timeout;
+
+       if (state->blob == NULL) {
+               state->result = true;
+               return;
+       }
+
+       *state->blob = data_blob(blob.data, blob.length);
+       if (state->blob->data == NULL) {
+               state->result = false;
+               return;
+       }
+       state->result = true;
 }
 
 /**
- * Delete one entry from the cache file.
+ * Get existing entry from the cache file.
  *
  * @param keystr string that represents a key of this entry
+ * @param blob DATA_BLOB that is filled with entry's blob
+ * @param timeout pointer to a time_t that is filled with entry's
+ *        timeout
  *
- * @retval true upon successful deletion
- * @retval false in case of failure
+ * @retval true when entry is successfuly fetched
+ * @retval False for failure
  **/
 
-BOOL gencache_del(const char *keystr)
+bool gencache_get_data_blob(const char *keystr, DATA_BLOB *blob,
+                           time_t *timeout, bool *was_expired)
 {
-       int ret;
-       TDB_DATA keybuf;
-       
-       /* fail completely if get null pointers passed */
-       SMB_ASSERT(keystr);
+       struct gencache_get_data_blob_state state;
+       bool expired = false;
 
-       if (!gencache_init()) return False;     
-       
-       keybuf.dptr = SMB_STRDUP(keystr);
-       keybuf.dsize = strlen(keystr)+1;
-       DEBUG(10, ("Deleting cache entry (key = %s)\n", keystr));
-       ret = tdb_delete(cache, keybuf);
-       
-       SAFE_FREE(keybuf.dptr);
-       return ret == 0;
+       state.result = false;
+       state.blob = blob;
+
+       if (!gencache_parse(keystr, gencache_get_data_blob_parser, &state)) {
+               goto fail;
+       }
+       if (!state.result) {
+               goto fail;
+       }
+       if (state.timeout <= time(NULL)) {
+               /*
+                * We're expired, delete the entry. We can't use gencache_del
+                * here, because that uses gencache_get_data_blob for checking
+                * the existence of a record. We know the thing exists and
+                * directly store an empty value with 0 timeout.
+                */
+               gencache_set(keystr, "", 0);
+               expired = true;
+               goto fail;
+       }
+       if (timeout) {
+               *timeout = state.timeout;
+       }
+
+       return True;
+
+fail:
+       if (was_expired != NULL) {
+               *was_expired = expired;
+       }
+       if (state.result && state.blob) {
+               data_blob_free(state.blob);
+       }
+       return false;
+} 
+
+struct stabilize_state {
+       bool written;
+       bool error;
+};
+static int stabilize_fn(struct tdb_context *tdb, TDB_DATA key, TDB_DATA val,
+                       void *priv);
+
+/**
+ * Stabilize gencache
+ *
+ * Migrate the clear-if-first gencache data to the stable,
+ * transaction-based gencache.tdb
+ */
+
+bool gencache_stabilize(void)
+{
+       struct stabilize_state state;
+       int res;
+       char *now;
+
+       if (!gencache_init()) {
+               return false;
+       }
+
+       res = tdb_transaction_start_nonblock(cache);
+       if (res == -1) {
+
+               if (tdb_error(cache) == TDB_ERR_NOLOCK) {
+                       /*
+                        * Someone else already does the stabilize,
+                        * this does not have to be done twice
+                        */
+                       return true;
+               }
+
+               DEBUG(10, ("Could not start transaction on gencache.tdb: "
+                          "%s\n", tdb_errorstr(cache)));
+               return false;
+       }
+       res = tdb_transaction_start(cache_notrans);
+       if (res == -1) {
+               tdb_transaction_cancel(cache);
+               DEBUG(10, ("Could not start transaction on "
+                          "gencache_notrans.tdb: %s\n",
+                          tdb_errorstr(cache_notrans)));
+               return false;
+       }
+
+       state.error = false;
+       state.written = false;
+
+       res = tdb_traverse(cache_notrans, stabilize_fn, &state);
+       if ((res == -1) || state.error) {
+               if ((tdb_transaction_cancel(cache_notrans) == -1)
+                   || (tdb_transaction_cancel(cache) == -1)) {
+                       smb_panic("tdb_transaction_cancel failed\n");
+               }
+               return false;
+       }
+
+       if (!state.written) {
+               if ((tdb_transaction_cancel(cache_notrans) == -1)
+                   || (tdb_transaction_cancel(cache) == -1)) {
+                       smb_panic("tdb_transaction_cancel failed\n");
+               }
+               return true;
+       }
+
+       res = tdb_transaction_commit(cache);
+       if (res == -1) {
+               DEBUG(10, ("tdb_transaction_commit on gencache.tdb failed: "
+                          "%s\n", tdb_errorstr(cache)));
+               if (tdb_transaction_cancel(cache_notrans) == -1) {
+                       smb_panic("tdb_transaction_cancel failed\n");
+               }
+               return false;
+       }
+
+       res = tdb_transaction_commit(cache_notrans);
+       if (res == -1) {
+               DEBUG(10, ("tdb_transaction_commit on gencache.tdb failed: "
+                          "%s\n", tdb_errorstr(cache)));
+               return false;
+       }
+
+       now = talloc_asprintf(talloc_tos(), "%d", (int)time(NULL));
+       if (now != NULL) {
+               tdb_store(cache_notrans, last_stabilize_key(),
+                         string_term_tdb_data(now), 0);
+               TALLOC_FREE(now);
+       }
+
+       return true;
 }
 
+static int stabilize_fn(struct tdb_context *tdb, TDB_DATA key, TDB_DATA val,
+                       void *priv)
+{
+       struct stabilize_state *state = (struct stabilize_state *)priv;
+       int res;
+       time_t timeout;
+
+       if (tdb_data_cmp(key, last_stabilize_key()) == 0) {
+               return 0;
+       }
+
+       if (!gencache_pull_timeout((char *)val.dptr, &timeout, NULL)) {
+               DEBUG(10, ("Ignoring invalid entry\n"));
+               return 0;
+       }
+       if ((timeout < time(NULL)) || (val.dsize == 0)) {
+               res = tdb_delete(cache, key);
+               if ((res == -1) && (tdb_error(cache) == TDB_ERR_NOEXIST)) {
+                       res = 0;
+               } else {
+                       state->written = true;
+               }
+       } else {
+               res = tdb_store(cache, key, val, 0);
+               if (res == 0) {
+                       state->written = true;
+               }
+       }
+
+       if (res == -1) {
+               DEBUG(10, ("Transfer to gencache.tdb failed: %s\n",
+                          tdb_errorstr(cache)));
+               state->error = true;
+               return -1;
+       }
+
+       if (tdb_delete(cache_notrans, key) == -1) {
+               DEBUG(10, ("tdb_delete from gencache_notrans.tdb failed: "
+                          "%s\n", tdb_errorstr(cache_notrans)));
+               state->error = true;
+               return -1;
+       }
+       return 0;
+}
 
 /**
  * Get existing entry from the cache file.
@@ -235,66 +590,131 @@ BOOL gencache_del(const char *keystr)
  * @retval False for failure
  **/
 
-BOOL gencache_get(const char *keystr, char **valstr, time_t *timeout)
+bool gencache_get(const char *keystr, char **value, time_t *ptimeout)
 {
-       TDB_DATA keybuf, databuf;
+       DATA_BLOB blob;
+       bool ret = False;
 
-       /* fail completely if get null pointers passed */
-       SMB_ASSERT(keystr);
-
-       if (!gencache_init())
-               return False;
-       
-       keybuf.dptr = SMB_STRDUP(keystr);
-       keybuf.dsize = strlen(keystr)+1;
-       databuf = tdb_fetch(cache, keybuf);
-       SAFE_FREE(keybuf.dptr);
-       
-       if (databuf.dptr && databuf.dsize > TIMEOUT_LEN) {
-               char* entry_buf = SMB_STRNDUP(databuf.dptr, databuf.dsize);
-               char *v;
-               time_t t;
-               unsigned u;
-               int status;
-
-               v = SMB_MALLOC(databuf.dsize - TIMEOUT_LEN);
-                               
-               SAFE_FREE(databuf.dptr);
-               status = sscanf(entry_buf, CACHE_DATA_FMT, &u, v);
-               if ( status != 2 ) {
-                   DEBUG(0, ("gencache_get: Invalid return %d from sscanf\n", status ));
+       ret = gencache_get_data_blob(keystr, &blob, ptimeout, NULL);
+       if (!ret) {
+               return false;
+       }
+       if ((blob.data == NULL) || (blob.length == 0)) {
+               SAFE_FREE(blob.data);
+               return false;
+       }
+       if (blob.data[blob.length-1] != '\0') {
+               /* Not NULL terminated, can't be a string */
+               SAFE_FREE(blob.data);
+               return false;
+       }
+       if (value) {
+               *value = SMB_STRDUP((char *)blob.data);
+               data_blob_free(&blob);
+               if (*value == NULL) {
+                       return false;
                }
-               t = u;
-               SAFE_FREE(entry_buf);
+               return true;
+       }
+       data_blob_free(&blob);
+       return true;
+}
 
-               DEBUG(10, ("Returning %s cache entry: key = %s, value = %s, "
-                          "timeout = %s", t > time(NULL) ? "valid" :
-                          "expired", keystr, v, ctime(&t)));
+/**
+ * Set an entry in the cache file. If there's no such
+ * one, then add it.
+ *
+ * @param keystr string that represents a key of this entry
+ * @param value text representation value being cached
+ * @param timeout time when the value is expired
+ *
+ * @retval true when entry is successfuly stored
+ * @retval false on failure
+ **/
 
-               if (valstr)
-                       *valstr = v;
-               else
-                       SAFE_FREE(v);
+bool gencache_set(const char *keystr, const char *value, time_t timeout)
+{
+       DATA_BLOB blob = data_blob_const(value, strlen(value)+1);
+       return gencache_set_data_blob(keystr, &blob, timeout);
+}
 
-               if (timeout)
-                       *timeout = t;
+struct gencache_iterate_blobs_state {
+       void (*fn)(const char *key, DATA_BLOB value,
+                  time_t timeout, void *private_data);
+       const char *pattern;
+       void *private_data;
+       bool in_persistent;
+};
 
-               return t > time(NULL);
+static int gencache_iterate_blobs_fn(struct tdb_context *tdb, TDB_DATA key,
+                                    TDB_DATA data, void *priv)
+{
+       struct gencache_iterate_blobs_state *state =
+               (struct gencache_iterate_blobs_state *)priv;
+       char *keystr;
+       char *free_key = NULL;
+       time_t timeout;
+       char *endptr;
+
+       if (tdb_data_cmp(key, last_stabilize_key()) == 0) {
+               return 0;
+       }
+       if (state->in_persistent && tdb_exists(cache_notrans, key)) {
+               return 0;
+       }
 
-       } 
+       if (key.dptr[key.dsize-1] == '\0') {
+               keystr = (char *)key.dptr;
+       } else {
+               /* ensure 0-termination */
+               keystr = SMB_STRNDUP((char *)key.dptr, key.dsize);
+               free_key = keystr;
+       }
 
-       SAFE_FREE(databuf.dptr);
+       if (!gencache_pull_timeout((char *)data.dptr, &timeout, &endptr)) {
+               goto done;
+       }
+       endptr += 1;
 
-       if (valstr)
-               *valstr = NULL;
-       if (timeout)
-               timeout = NULL;
+       if (fnmatch(state->pattern, keystr, 0) != 0) {
+               goto done;
+       }
+
+       DEBUG(10, ("Calling function with arguments (key=%s, timeout=%s)\n",
+                  keystr, ctime(&timeout)));
 
-       DEBUG(10, ("Cache entry with key = %s couldn't be found\n", keystr));
+       state->fn(keystr,
+                 data_blob_const(endptr,
+                                 data.dsize - PTR_DIFF(endptr, data.dptr)),
+                 timeout, state->private_data);
 
-       return False;
+ done:
+       SAFE_FREE(free_key);
+       return 0;
 }
 
+void gencache_iterate_blobs(void (*fn)(const char *key, DATA_BLOB value,
+                                      time_t timeout, void *private_data),
+                           void *private_data, const char *pattern)
+{
+       struct gencache_iterate_blobs_state state;
+
+       if ((fn == NULL) || (pattern == NULL) || !gencache_init()) {
+               return;
+       }
+
+       DEBUG(5, ("Searching cache keys with pattern %s\n", pattern));
+
+       state.fn = fn;
+       state.pattern = pattern;
+       state.private_data = private_data;
+
+       state.in_persistent = false;
+       tdb_traverse(cache_notrans, gencache_iterate_blobs_fn, &state);
+
+       state.in_persistent = true;
+       tdb_traverse(cache, gencache_iterate_blobs_fn, &state);
+}
 
 /**
  * Iterate through all entries which key matches to specified pattern
@@ -307,85 +727,47 @@ BOOL gencache_get(const char *keystr, char **valstr, time_t *timeout)
  *
  **/
 
-void gencache_iterate(void (*fn)(const char* key, const char *value, time_t timeout, void* dptr),
-                      void* data, const char* keystr_pattern)
+struct gencache_iterate_state {
+       void (*fn)(const char *key, const char *value, time_t timeout,
+                  void *priv);
+       void *private_data;
+};
+
+static void gencache_iterate_fn(const char *key, DATA_BLOB value,
+                               time_t timeout, void *private_data)
 {
-       TDB_LIST_NODE *node, *first_node;
-       TDB_DATA databuf;
-       char *keystr = NULL, *valstr = NULL, *entry = NULL;
-       time_t timeout = 0;
-       int status;
-       unsigned u;
-
-       /* fail completely if get null pointers passed */
-       SMB_ASSERT(fn && keystr_pattern);
-
-       if (!gencache_init()) return;
-
-       DEBUG(5, ("Searching cache keys with pattern %s\n", keystr_pattern));
-       node = tdb_search_keys(cache, keystr_pattern);
-       first_node = node;
-       
-       while (node) {
-               /* ensure null termination of the key string */
-               keystr = SMB_STRNDUP(node->node_key.dptr, node->node_key.dsize);
-               
-               /* 
-                * We don't use gencache_get function, because we need to iterate through
-                * all of the entries. Validity verification is up to fn routine.
-                */
-               databuf = tdb_fetch(cache, node->node_key);
-               if (!databuf.dptr || databuf.dsize <= TIMEOUT_LEN) {
-                       SAFE_FREE(databuf.dptr);
-                       SAFE_FREE(keystr);
-                       node = node->next;
-                       continue;
-               }
-               entry = SMB_STRNDUP(databuf.dptr, databuf.dsize);
-               SAFE_FREE(databuf.dptr);
-               valstr = SMB_MALLOC(databuf.dsize - TIMEOUT_LEN);
-               status = sscanf(entry, CACHE_DATA_FMT, &u, valstr);
-               if ( status != 2 ) {
-                   DEBUG(0,("gencache_iterate: invalid return from sscanf %d\n",status));
-               }
-               timeout = u;
-               
-               DEBUG(10, ("Calling function with arguments (key = %s, value = %s, timeout = %s)\n",
-                          keystr, valstr, ctime(&timeout)));
-               fn(keystr, valstr, timeout, data);
-               
-               SAFE_FREE(valstr);
-               SAFE_FREE(entry);
-               SAFE_FREE(keystr);
-               node = node->next;
-       }
-       
-       tdb_search_list_free(first_node);
-}
+       struct gencache_iterate_state *state =
+               (struct gencache_iterate_state *)private_data;
+       char *valstr;
+       char *free_val = NULL;
+
+       if (value.data[value.length-1] == '\0') {
+               valstr = (char *)value.data;
+       } else {
+               /* ensure 0-termination */
+               valstr = SMB_STRNDUP((char *)value.data, value.length);
+               free_val = valstr;
+       }
 
-/********************************************************************
- lock a key
-********************************************************************/
+       DEBUG(10, ("Calling function with arguments "
+                  "(key = %s, value = %s, timeout = %s)\n",
+                  key, valstr, ctime(&timeout)));
 
-int gencache_lock_entry( const char *key )
-{
-       if (!gencache_init())
-               return -1;
-       
-       return tdb_lock_bystring(cache, key, 0);
-}
+       state->fn(key, valstr, timeout, state->private_data);
 
-/********************************************************************
- unlock a key
-********************************************************************/
+       SAFE_FREE(free_val);
+}
 
-void gencache_unlock_entry( const char *key )
+void gencache_iterate(void (*fn)(const char *key, const char *value,
+                                time_t timeout, void *dptr),
+                      void *private_data, const char *pattern)
 {
-       if (!gencache_init())
+       struct gencache_iterate_state state;
+
+       if (fn == NULL) {
                return;
-       
-       tdb_unlock_bystring(cache, key);
-       return;
+       }
+       state.fn = fn;
+       state.private_data = private_data;
+       gencache_iterate_blobs(gencache_iterate_fn, &state, pattern);
 }
-
-