[all-commits] [llvm/llvm-project] e89458: [lldb] Fix PipeTest name collision in unit tests

[Jonas Devlieghere via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e89458d3985c1b612b8a64914c887a3ce3dd3509
      https://github.com/llvm/llvm-project/commit/e89458d3985c1b612b8a64914c887a3ce3dd3509
  Author: Jonas Devlieghere <jonas at devlieghere.com>
  Date:   2025-07-01 (Tue, 01 Jul 2025)

  Changed paths:
    M lldb/unittests/DAP/TestBase.cpp
    M lldb/unittests/DAP/TestBase.h
    M lldb/unittests/Host/JSONTransportTest.cpp
    M lldb/unittests/TestingSupport/Host/PipeTestUtilities.h

  Log Message:
  -----------
  [lldb] Fix PipeTest name collision in unit tests

We had two classes named `PipeTest`: one in `PipeTestUtilities.h` and
one in `PipeTest.cpp`. The latter was unintentionally using the wrong
class (from the header) which didn't initialize the HostInfo subsystem.

This resulted in a crash due to a nullptr dereference (`g_fields`) when
`PipePosix::CreateWithUniqueName` called `HostInfoBase::GetProcessTempDir`.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications