Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix NPE in ArrayByteBufferPool.removeAndRelease() when the RBB comes from QueuedPool #12799

Open
wants to merge 2 commits into
base: jetty-12.0.x
Choose a base branch
from
Open

Fix NPE in ArrayByteBufferPool.removeAndRelease() when the RBB comes from QueuedPool #12799

wants to merge 2 commits into from

Conversation

lorban
Copy link
Contributor

@lorban lorban commented Feb 17, 2025 •
edited
Loading

Fixes #12790

@lorban
#12790 add reproducer
e76c3ee 
Signed-off-by: Ludovic Orban <[email protected]>
@lorban lorban added the Bug For general bugs on Jetty side label Feb 17, 2025
@lorban lorban self-assigned this Feb 17, 2025
@lorban lorban mentioned this pull request Feb 17, 2025
Open
@lorban
#12790 quick fix
63425f2 
Signed-off-by: Ludovic Orban <[email protected]>
@lorban lorban requested review from sbordet and gregw February 17, 2025 15:42
@lorban
Copy link
Contributor Author

lorban commented Feb 21, 2025

@gregw @sbordet nudge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sbordet sbordet Awaiting requested review from sbordet

@gregw gregw Awaiting requested review from gregw

At least 1 approving review is required to merge this pull request.

Assignees

@lorban lorban

Labels
Bug For general bugs on Jetty side
Projects
Jetty 12.0.18
Status: 🏗 In progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Cannot invoke "org.eclipse.jetty.io.RetainableByteBuffer.getByteBuffer()" because "buffer" is null
1 participant
@lorban