mirror of
https://github.com/Kitware/CMake.git
synced 2026-08-09 09:12:00 +00:00
194232e2bc
The POSIX pipe read wrappers stored the return of ::read() into an unsigned result, so a negative count (e.g. EBADF from a concurrent close on another thread) became SIZE_MAX. ContentReader::buffer() would then try to grow its deque by ~18 exabytes and crash with either std::length_error (glibc) or a stack smash (libc++). Treat any non-positive ::read() return as EOF/error, close the pipe, and return 0 so the peer's SessionThread observes a clean empty payload. cmDebuggerPipeClient is test-only infrastructure; production cmake is always the pipe server. Production close() assumes sequential access, which holds in normal use (the adapter destructor joins SessionThread before closing the connection). The new abrupt- disconnect test however needs to wake a sibling thread blocked in read() on the same fd, which Linux ::close() does not do. Add a ShutdownForTesting() method that calls shutdown(SHUT_RDWR) without freeing the fd, so any concurrent blocking read wakes with a clean zero-length return. On Windows, CloseHandle already cancels pending overlapped I/O, so the helper just forwards to close(). Use it from testProtocolWithPipesAbruptDisconnect in place of close().
87 lines
2.2 KiB
C++
87 lines
2.2 KiB
C++
/* Distributed under the OSI-approved BSD 3-Clause License. See accompanying
|
|
file LICENSE.rst or https://cmake.org/licensing for details. */
|
|
#pragma once
|
|
|
|
#include "cmConfigure.h" // IWYU pragma: keep
|
|
|
|
#include <cstddef>
|
|
#include <future>
|
|
#include <memory>
|
|
#include <string>
|
|
|
|
#include <cm3p/cppdap/io.h>
|
|
|
|
#include <sys/un.h>
|
|
|
|
#include "cmDebuggerAdapter.h"
|
|
|
|
namespace cmDebugger {
|
|
|
|
#ifndef _WIN32
|
|
|
|
class cmDebuggerPipeConnection_POSIX
|
|
: public dap::ReaderWriter
|
|
, public cmDebuggerConnection
|
|
, public std::enable_shared_from_this<cmDebuggerPipeConnection_POSIX>
|
|
{
|
|
public:
|
|
cmDebuggerPipeConnection_POSIX(std::string name);
|
|
~cmDebuggerPipeConnection_POSIX() override;
|
|
|
|
void WaitForConnection() override;
|
|
|
|
bool StartListening(std::string& errorMessage) override;
|
|
std::shared_ptr<dap::Reader> GetReader() override;
|
|
std::shared_ptr<dap::Writer> GetWriter() override;
|
|
|
|
// dap::ReaderWriter implementation
|
|
|
|
bool isOpen() override;
|
|
void close() override;
|
|
size_t read(void* buffer, size_t n) override;
|
|
bool write(void const* buffer, size_t n) override;
|
|
|
|
// Used for unit test synchronization
|
|
std::promise<void> StartedListening;
|
|
|
|
private:
|
|
void close_listen(); // release listen resources
|
|
|
|
std::string const PipeName;
|
|
sockaddr_un addr;
|
|
int listen_fd = -1; // listen fd
|
|
int rw_pipe = -1; // rw fd
|
|
};
|
|
|
|
using cmDebuggerPipeConnection = cmDebuggerPipeConnection_POSIX;
|
|
|
|
class cmDebuggerPipeClient_POSIX
|
|
: public dap::ReaderWriter
|
|
, public std::enable_shared_from_this<cmDebuggerPipeClient_POSIX>
|
|
{
|
|
public:
|
|
cmDebuggerPipeClient_POSIX(std::string name);
|
|
~cmDebuggerPipeClient_POSIX() override;
|
|
void WaitForConnection();
|
|
|
|
bool isOpen() override;
|
|
void close() override;
|
|
size_t read(void* buffer, size_t n) override;
|
|
bool write(void const* buffer, size_t n) override;
|
|
|
|
// Unit-test helper: signal EOF on both halves of the socket without
|
|
// freeing the fd, so a sibling thread blocked in read() wakes up
|
|
// cleanly. Production code does not use cmDebuggerPipeClient.
|
|
void ShutdownForTesting();
|
|
|
|
private:
|
|
std::string const PipeName;
|
|
int rw_pipe = -1;
|
|
};
|
|
|
|
using cmDebuggerPipeClient = cmDebuggerPipeClient_POSIX;
|
|
|
|
#endif // !_WIN32
|
|
|
|
} // namespace cmDebugger
|