Merge pull request #5866 from daira/revert-span-reload

Partially revert #4909, in order to avoid a use-after-free after calling setlogfilter
This commit is contained in:
Charlie O'Keefe 2022-04-12 09:00:26 -06:00 committed by GitHub
commit 6205075970
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 9 deletions

View File

@ -8,7 +8,6 @@
#include "fs.h"
#include "init.h"
#include "key_io.h"
#include "net.h"
#include "random.h"
#include "sync.h"
#include "ui_interface.h"
@ -274,14 +273,6 @@ UniValue setlogfilter(const UniValue& params, bool fHelp)
if (!tracing_reload(pTracingHandle, newFilter.c_str())) {
throw JSONRPCError(RPC_INTERNAL_ERROR, "Filter reload failed; check logs");
}
// Now that we have reloaded the filter, reload any stored spans.
{
LOCK(cs_vNodes);
for (CNode* pnode : vNodes) {
pnode->ReloadTracingSpan();
}
}
}
return NullUniValue;