Skip to content

[pull] unstable from redis:unstable#64

Merged
pull[bot] merged 1 commit intodolfly:unstablefrom
redis:unstable
Sep 8, 2025
Merged

[pull] unstable from redis:unstable#64
pull[bot] merged 1 commit intodolfly:unstablefrom
redis:unstable

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Sep 8, 2025

See Commits and Changes for more details.


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

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

This PR fixes three defrag issues.

1. Fix the issue that forget to update cgroup_ref_node when the consume
group was reallocated.
This crash was introduced by #14130
In this PR, when performing defragmentation on `s->cgroups` using
`defragRadixTree()`, we no longer rely on the automatic data
defragmentation of `defragRadixTree()`. Instead, we manually defragment
the consumer group and then update its reference in `s->cgroups`.

2. Fix a use-after-free issue caused by updating dictionary keys after
HFE key is reallocated.
This issue was introduced by #13842

3. Fix the issue that forgot to be updated NextSegHdr->firstSeg when the
first segment was reallocated.
This issue was introduced by #13842

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@pull pull bot locked and limited conversation to collaborators Sep 8, 2025
@pull pull bot added the ⤵️ pull label Sep 8, 2025
@pull pull bot merged commit d649637 into dolfly:unstable Sep 8, 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.

1 participant