diff options
| author | Nick Khyl <nickk@tailscale.com> | 2026-04-07 12:40:25 -0500 |
|---|---|---|
| committer | Nick Khyl <1761190+nickkhyl@users.noreply.github.com> | 2026-04-07 12:55:11 -0500 |
| commit | 1f84729908affe91ab650eb3f86b0d7263f6115a (patch) | |
| tree | aa0b0671cf00cb78bc564514321281ce12fe481d /control/controlhttp/controlhttpserver/controlhttpserver.go | |
| parent | 1b5b43787c33a197977a92d108297cac9b090ddf (diff) | |
| download | tailscale-1f84729908affe91ab650eb3f86b0d7263f6115a.tar.xz tailscale-1f84729908affe91ab650eb3f86b0d7263f6115a.zip | |
ipn/desktop: use runtime.Pinner to force heap-allocation of msg
GetMessage can call back into Go, triggering stack growth and causing the stack
to be copied to a new memory region, which invalidates the original stack pointer
passed to the syscall. Since GetMessage uses that pointer to write the message
before returning, this leads to memory corruption.
In this PR, we fix this by using runtime.Pinner, which requires the pointer to refer
to heap-allocated memory.
Fixes #19263
Fixes #17832
Signed-off-by: Nick Khyl <nickk@tailscale.com>
Diffstat (limited to 'control/controlhttp/controlhttpserver/controlhttpserver.go')
0 files changed, 0 insertions, 0 deletions