Skip to content

Make sure the sentinel Magazine.MAGAZINE_FREED not be replaced#14502

Merged
normanmaurer merged 1 commit intonetty:4.1from
laosijikaichele:4.1-MAGAZINE_FREED
Nov 26, 2024
Merged

Make sure the sentinel Magazine.MAGAZINE_FREED not be replaced#14502
normanmaurer merged 1 commit intonetty:4.1from
laosijikaichele:4.1-MAGAZINE_FREED

Conversation

@laosijikaichele
Copy link
Copy Markdown
Contributor

Motivation:

The sentinel object Magazine.MAGAZINE_FREED should not be replaced.

Modification:

Check the NEXT_IN_LINE to make sure the Magazine.MAGAZINE_FREED not be replaced.

Result:

Fixes #14498.

Copy link
Copy Markdown
Member

@chrisvest chrisvest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this is what it should've been from the start.

@normanmaurer normanmaurer added this to the 4.1.116.Final milestone Nov 26, 2024
@normanmaurer normanmaurer merged commit 3d2b816 into netty:4.1 Nov 26, 2024
normanmaurer pushed a commit that referenced this pull request Nov 26, 2024
)

Motivation:

The sentinel object `Magazine.MAGAZINE_FREED` should not be replaced.

Modification:

Check the `NEXT_IN_LINE` to make sure the `Magazine.MAGAZINE_FREED` not
be replaced.

Result:

Fixes #14498.

Co-authored-by: laosijikaichele <laosijikaichele>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sentinel object Magazine.MAGAZINE_FREED is possible be replaced without be restored back

3 participants