Skip to content

[pull] main from facebook:main#1193

Merged
pull[bot] merged 4 commits intoMu-L:mainfrom
facebook:main
Nov 10, 2025
Merged

[pull] main from facebook:main#1193
pull[bot] merged 4 commits intoMu-L:mainfrom
facebook:main

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Nov 10, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

Open Source Bot and others added 4 commits November 10, 2025 09:36
Summary:
Pull Request resolved: #583

Noticed these are showing up as stale on each regeneration

Reviewed By: bigfootjon

Differential Revision: D86594247

fbshipit-source-id: 2d4e361b7186d16915a0a6af99fe7812171f03b4
Summary:
`StaticFileHandler` has several race conditions arising from the fact that two different threads (CpuExecutor via `::getUnsafeMutableGlobalCPUExecutor` & `EventBase` backing the HttpSession) are accessing the same memory

A very simple sequence of events that could cause a crash
1. client sends `GET /index.html`
2. server begins serving bytes from the file
3. client sends malformed quic frame, causing all `HttpTransaction`s to abort and be destructed
4. reading chunk from file ends, attempt to use HttpTransaction (uaf)

Reviewed By: jbeshay

Differential Revision: D86682244

fbshipit-source-id: 9058e826c9df667656848cd63005924e7bb74eac
Summary:
Fix edencommon build on windows, noticed it was breaking the watchman build.

  *  fix boost local build with vs build tools 2022 17.14 update. Without this could not build boost locally. Apply patch as per boostorg/boost#914
  *  in buildopts.py check the 32 bit install location as non-github windows machines like my laptop end up with it installed there
  *  fix the actual problem in edencommon `error C2039: 'wstring': is not a member of 'std'` , which was a missing <string> include

Reviewed By: bigfootjon

Differential Revision: D86630363

fbshipit-source-id: b4993c74cd51d326efb458a6dc461d355f8daeab
@pull pull bot locked and limited conversation to collaborators Nov 10, 2025
@pull pull bot added the ⤵️ pull label Nov 10, 2025
@pull pull bot merged commit a8c44a7 into Mu-L:main Nov 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants