Commit 86028553 authored by Rusty Russell's avatar Rusty Russell

tdb2: don't use probe arg to oob()

We want it to complain if it's wrong...
parent 96964cc8
...@@ -287,8 +287,8 @@ static bool check_free(struct tdb_context *tdb, ...@@ -287,8 +287,8 @@ static bool check_free(struct tdb_context *tdb,
return false; return false;
} }
if (tdb->methods->oob(tdb, off if (tdb->methods->oob(tdb, off
+ frec->data_len-sizeof(struct tdb_used_record), + frec->data_len+sizeof(struct tdb_used_record),
true)) false))
return false; return false;
if (off < zone_off || off >= zone_off + (1ULL<<frec_zone_bits(frec))) { if (off < zone_off || off >= zone_off + (1ULL<<frec_zone_bits(frec))) {
tdb->log(tdb, TDB_DEBUG_ERROR, tdb->log_priv, tdb->log(tdb, TDB_DEBUG_ERROR, tdb->log_priv,
...@@ -487,7 +487,6 @@ int tdb_check(struct tdb_context *tdb, ...@@ -487,7 +487,6 @@ int tdb_check(struct tdb_context *tdb,
unsigned max_zone_bits = INITIAL_ZONE_BITS; unsigned max_zone_bits = INITIAL_ZONE_BITS;
uint8_t tailer; uint8_t tailer;
/* This always ensures the header is uptodate. */
if (tdb_allrecord_lock(tdb, F_RDLCK, TDB_LOCK_WAIT, false) != 0) if (tdb_allrecord_lock(tdb, F_RDLCK, TDB_LOCK_WAIT, false) != 0)
return -1; return -1;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment