Skip to content

ESQL: Fix IP_PREFIX function leaking data in scratch#141940

Merged
ivancea merged 2 commits intoelastic:mainfrom
ivancea:esql-ip-prefix-leak-bug
Feb 5, 2026
Merged

ESQL: Fix IP_PREFIX function leaking data in scratch#141940
ivancea merged 2 commits intoelastic:mainfrom
ivancea:esql-ip-prefix-leak-bug

Conversation

@ivancea
Copy link
Contributor

@ivancea ivancea commented Feb 5, 2026

Fixes #141628

There's a case where one byte in the IP_PREFIX scratch isn't cleaned up, and its dirty data gets carried into the generated IP. It's better explained in the linked issue.

Thanks @strawgate for looking into it and identifying the exact bug!

@ivancea ivancea requested review from MattAlp and strawgate February 5, 2026 15:50
@ivancea ivancea added >bug Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) auto-backport Automatically create backport pull requests when merged :Analytics/ES|QL AKA ESQL v9.3.1 v9.4.0 v8.19.12 v9.2.6 labels Feb 5, 2026
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@elasticsearchmachine
Copy link
Collaborator

Hi @ivancea, I've created a changelog YAML for you.

@ivancea ivancea merged commit c8eb975 into elastic:main Feb 5, 2026
35 checks passed
@ivancea ivancea deleted the esql-ip-prefix-leak-bug branch February 5, 2026 17:26
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
9.3
8.19 Commit could not be cherrypicked due to conflicts
9.2 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 141940

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

Labels

:Analytics/ES|QL AKA ESQL auto-backport Automatically create backport pull requests when merged >bug Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.19.12 v9.2.6 v9.3.1 v9.4.0

4 participants