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

Guard against negative Length in ssl:recv/3 #7526

Conversation

RaimoNiskanen
Copy link
Contributor

Solves #7507

@RaimoNiskanen RaimoNiskanen added team:PS Assigned to OTP team PS enhancement in progress priority:medium types The issue is related to types labels Aug 2, 2023
@RaimoNiskanen RaimoNiskanen added this to the OTP-26.1 milestone Aug 2, 2023
@RaimoNiskanen RaimoNiskanen self-assigned this Aug 2, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Aug 2, 2023

CT Test Results

       2 files       66 suites   49m 1s ⏱️
   770 tests    726 ✔️   44 💤 0
3 663 runs  2 908 ✔️ 755 💤 0

Results for commit c14f97b.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@RaimoNiskanen RaimoNiskanen changed the base branch from master to maint August 2, 2023 14:40
@RaimoNiskanen RaimoNiskanen merged commit a01865b into erlang:maint Aug 4, 2023
13 of 16 checks passed
@RaimoNiskanen
Copy link
Contributor Author

I have also backported this fix to be released in OTP-25.3.2.6, whenever that is due...

@RaimoNiskanen RaimoNiskanen deleted the raimo/ssl/non_neg_integer-length/GH-7507/OTP-18700 branch August 4, 2023 08:36
@RaimoNiskanen RaimoNiskanen linked an issue Aug 4, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement in progress priority:medium team:PS Assigned to OTP team PS types The issue is related to types
Projects
None yet
Development

Successfully merging this pull request may close these issues.

{case_clause,[]} in ssl_gen_statem:iovec_from_front/4
2 participants