It seems that it would otherwise silently skip the rescan for pre-backup used keys that are still waiting to be generated into the keypool?

See also #11249 
}
    DatabaseCursorStatus status = DatabaseCursorStatusFAIL;
    while (true) {
        DataStream ss_key{};
        DataStream ss_value{};
        status = cursor->Next(ss_key, ss_val
        if (status != DatabaseCursorStatusMORE) {
            break;
        }
        SerializeData key(ss_key.begin(), ss_key.end());
        SerializeData value(ss_value.begin(), ss_value.end());
        records.emplace_back(key, val
    }
    cursor.reset();
    batch.reset();
    if (status != DatabaseCursorStatusDONE) {
        error = _("Error Unable to read all records in the database");
        return false;
    }
    // Close this database and delete the file
    fspath db_path = fsPathFromString(m_database->Filename());
    fspath db_dir = db_path.parent_path();
    m_database->Close();
    fsremove(db_path);
    // Make new DB
    DatabaseOptions opts;
    opts.require_create = true;
    opts.require_format = DatabaseFormatSQLITE;
    DatabaseStatus db_status;
    stdunique_ptr<WalletDatabase> new_db = MakeDatabase(db_dir, opts, db_status, error);