[Lldb-commits] [PATCH] D86388: Fix use-after-free in ThreadPlan, and add test.

[Jim Ingham via Phabricator via lldb-commits]

jingham added a comment.

I'm confused as to how this patch actually fixes the problem.  When the thread gets removed from the thread list, it should get Destroy called on it - which should set m_destroy_called, causing IsValid to return false..  So I am not clear under what circumstances FindThreadByID will fail, but the cached thread shared pointer's IsValid is still true?  If IsValid is holding true over the thread's removal from the thread list, then I'm worried that this change will keep us using the old ThreadSP that was reported the next time we stopped and this thread ID was represented by a different ThreadSP.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86388/new/

https://reviews.llvm.org/D86388