Skip to content

fix: prevent UB stress transfer lifetime races#16

Open
YChange01 wants to merge 1 commit into
zchuango:supercache_devfrom
YChange01:fix/ub-stress-coredump
Open

fix: prevent UB stress transfer lifetime races#16
YChange01 wants to merge 1 commit into
zchuango:supercache_devfrom
YChange01:fix/ub-stress-coredump

Conversation

@YChange01

Copy link
Copy Markdown

Summary

  • keep an owned TransferRequest copy in each TransferTask instead of pointing at caller-owned vectors
  • defer UB successful completion publishing until the worker has finished local slice accounting
  • add a regression check for TransferTask request ownership

Test Plan

  • git diff --check
  • cmake -S . -B /private/tmp/mooncake-build -DBUILD_UNIT_TESTS=ON -DWITH_STORE=OFF -DWITH_STORE_RUST=OFF -DWITH_TE=ON -DUSE_ETCD=OFF -DUSE_HTTP=OFF -DUSE_REDIS=OFF (blocked locally: missing yalantinglibsConfig.cmake; extern/yalantinglibs submodule is not initialized)

@zchuango zchuango force-pushed the supercache_dev branch 2 times, most recently from 41943c4 to ccf5499 Compare June 3, 2026 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant