### Is there an existing issue for this?

- [X] I have searched the existing issues

### Current behaviour

crash

### Expected behaviour

no crash

### Steps to reproduce

* Compile 79e8247ddb166f9b980f40249b7372a502402a4d
* Type `migratewallet`

### Relevant log output

```
wallet/scriptpubkeyman.cpp:1915: std::optional<MigrationData> wallet::LegacyScriptPubKeyMan::MigrateToDescriptor(): Assertion `IsMine(desc_spk) != ISMINE_NO' failed.
Aborted (core dumped)
```


### How did you obtain Bitcoin Core

Compiled from source

### What version of Bitcoin Core are you using?

current master

### Operating system and version

Linux

### Machine specifications

_No response_
// wtx can only be null if the db write failed.
    if (!wtx) {
        throw stdruntime_error(stdstring(__func__) + " Wal db error, transaction commit failed");
    }
    // Notify that old coins are s
    for (const CTxIn& txin  tx->vin) {
        CWalTx &coin = mapWal.at(txin.prevout.hash);
        coin.MarkDirty();
        NotifyTransactionChanged(coin.GetHash(), CT_UPDATED);
    }
    if (!fBroadcastTransactions) {
        // Don't submit tx to the mempool
        return;
    }
    stdstring err_string;
    if (!SubmitTxMemoryPoolAndRelay(*wtx, err_string, true)) {
        WalLogPrintf("CommitTransaction() Transaction cannot be broadcast immediately, %s\n", err_string);
        // TODO if we expect the failure to be long term or permanent, instead dee wtx from th and return failure.
    }
}
DBErrors CWalLoadWal()
{
    LOCK(cs_wal);
    DBErrors nLoadWalRet = WalBatch(GetDatabase()).LoadWal(this);
    if (nLoadWalRet == DBErrorsNEED_REWRITE)
    {
        if (GetDatabase().Rewrite("\x04pool"))
        {
            for (const auto& spk_man_pair  m_spk_managers) {
                spk_man_pair.second->RewriteDB();