Skip to content

[8.5] Fix RecyclerBytesStreamOutput allocating unlimited heap for some capacities (#90632)#90636

Merged
elasticsearchmachine merged 1 commit intoelastic:8.5from
original-brownbear:backport/8.5/pr-90632
Oct 4, 2022
Merged

[8.5] Fix RecyclerBytesStreamOutput allocating unlimited heap for some capacities (#90632)#90636
elasticsearchmachine merged 1 commit intoelastic:8.5from
original-brownbear:backport/8.5/pr-90632

Conversation

@original-brownbear
Copy link
Copy Markdown
Contributor

Backports the following commits to 8.5:

…cities (#90632)

We'd allocate infinite memory here because we'd overflow to negative
capacities for newPosition close to `Integer.MAX_VALUE`.
@original-brownbear original-brownbear added :Distributed/Network Http and internode communication implementations >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Distributed Meta label for distributed team. labels Oct 4, 2022
@elasticsearchmachine elasticsearchmachine merged commit 154292b into elastic:8.5 Oct 4, 2022
@original-brownbear original-brownbear deleted the backport/8.5/pr-90632 branch October 4, 2022 10:04
@csoulios csoulios added v8.5.0 and removed v8.5.1 labels Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug :Distributed/Network Http and internode communication implementations Team:Distributed Meta label for distributed team. v8.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants