Cairn

TOCTOU race conditions in lazy-init globals

NousResearch/hermes-agent · cluster 24623 · agent

Data as of

3 open items. Fix attempted, not merged.

The short answer

2open issues
1open PR
8items grouped
Known?
Yes. 8 related items grouped here, 0 issues already closed.
Fix event
No fix has been merged to main. 5 PRs closed without merging; see the attempts below.
Tracker
The maintainers' tracker marks this unresolved. That is separate from the fix event.
Still open
2 issues and 1 PR.

Multiple issues and fixes targeting time-of-check-to-time-of-use race conditions in lazy initialization routines, particularly around _global_allow_private_urls and _load_google_modules(). The concern is that flags are set before their associated values or imports are fully ready, exposing concurrent callers to stale state.

Open issues2

  • Issue#24623[Bug]: _global_allow_private_urls() sets _allow_private_resolved=True before computing the cache value — concurrent callers see stale False
  • Issue#24673concurrency: _load_google_modules() TOCTOU — _google_modules_loaded set before imports complete

Open PRs1

  • PR#24636fix(url_safety): close TOCTOU race in _global_allow_private_urls (#24623)

5 PRs closed without merging

From the close comments: closed without comment 1 · done elsewhere 1 · duplicate 1 · superseded by another PR 1 · other 1

  • #24641 fix(url_safety): fix TOCTOU race in _global_allow_private_urls lazy-init · duplicate → #24636, #24623
  • #24645 fix(url_safety): eliminate TOCTOU race in _global_allow_private_urls · superseded by another PR → #24641
  • #24679 fix(google_chat): double-checked locking for _load_google_modules() TOCTOU · closed without comment
  • #24772 fix(core): close TOCTOU race windows in lazy module resolvers · done elsewhere → #24671, #24669, #24692, #24687, #24694, #24679, #24673
  • #128340 fix(google-chat): serialize lazy dependency initialization · other
How this cluster was grouped
  • Link evidence (partly structural links): some members are connected by structural evidence (shared changed files, or a closing/superseded reference); the rest are joined by an automated relevance judgement over their text.
  • Members outside that structural core may belong to a different underlying problem.
  • Pairwise link density 0.23: the share of item pairs that carry a link, counting a structural link in full and a relevance-only link at 0.36. 1.00 means every pair is linked.

Pick up a Lab task

Check whether a fix covers the open issues here, with your own agent. Bring back evidence for maintainers.

Open the Lab

At a glance

Subsystem
agent
Closed issues
0
Attempts
5 PRs closed without merging

For your agent

The same facts, typed.

cluster/24623.json
cluster/24623.md