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.

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

[v13] Prevent overflow in sort functions provided to slices.SortFunc #38678

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

rosstimothy
Copy link
Contributor

Backport #38642 to branch/v13

@rosstimothy rosstimothy added backport no-changelog Indicates that a PR does not require a changelog entry labels Feb 27, 2024
Replaces subtraction based comparisions with calls to cmp.Compare
for all ordered types to avoid overflows.
@rosstimothy rosstimothy force-pushed the tross/backport-38642/v13 branch from 4b0b53d to 77a312b Compare February 27, 2024 16:59
@rosstimothy rosstimothy marked this pull request as ready for review February 27, 2024 18:02
@github-actions github-actions bot requested review from codingllama and zmb3 February 27, 2024 18:02
@github-actions github-actions bot added audit-log Issues related to Teleports Audit Log size/sm labels Feb 27, 2024
@rosstimothy rosstimothy added this pull request to the merge queue Feb 27, 2024
Merged via the queue into branch/v13 with commit d4aa092 Feb 27, 2024
27 checks passed
@rosstimothy rosstimothy deleted the tross/backport-38642/v13 branch February 27, 2024 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
audit-log Issues related to Teleports Audit Log backport no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants