Skip to content

Comments

Fix wrong buffer and version in outgoing msg callback#28816

Closed
MegaManSec wants to merge 1 commit intoopenssl:masterfrom
MegaManSec:d1
Closed

Fix wrong buffer and version in outgoing msg callback#28816
MegaManSec wants to merge 1 commit intoopenssl:masterfrom
MegaManSec:d1

Conversation

@MegaManSec
Copy link
Contributor

No description provided.

@mattcaswell mattcaswell added branch: master Applies to master branch approval: review pending This pull request needs review by a committer triaged: bug The issue/pr is/fixes a bug branch: 3.0 Applies to openssl-3.0 branch tests: exempted The PR is exempt from requirements for testing branch: 3.2 Applies to openssl-3.2 (EOL) branch: 3.3 Applies to openssl-3.3 branch: 3.4 Applies to openssl-3.4 branch: 3.5 Applies to openssl-3.5 branch: 3.6 Applies to openssl-3.6 labels Oct 13, 2025
Copy link
Contributor

@Sashan Sashan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if Matt is happy I'm happy too. thanks.

@Sashan Sashan added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Oct 14, 2025
@openssl-machine openssl-machine added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Oct 15, 2025
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

openssl-machine pushed a commit that referenced this pull request Oct 17, 2025
Signed-off-by: Joshua Rogers <[email protected]>

Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Saša Nedvědický <[email protected]>
(Merged from #28816)
openssl-machine pushed a commit that referenced this pull request Oct 17, 2025
Signed-off-by: Joshua Rogers <[email protected]>

Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Saša Nedvědický <[email protected]>
(Merged from #28816)

(cherry picked from commit f7c0fcf)
openssl-machine pushed a commit that referenced this pull request Oct 17, 2025
Signed-off-by: Joshua Rogers <[email protected]>

Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Saša Nedvědický <[email protected]>
(Merged from #28816)

(cherry picked from commit f7c0fcf)
openssl-machine pushed a commit that referenced this pull request Oct 17, 2025
Signed-off-by: Joshua Rogers <[email protected]>

Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Saša Nedvědický <[email protected]>
(Merged from #28816)

(cherry picked from commit f7c0fcf)
openssl-machine pushed a commit that referenced this pull request Oct 17, 2025
Signed-off-by: Joshua Rogers <[email protected]>

Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Saša Nedvědický <[email protected]>
(Merged from #28816)

(cherry picked from commit f7c0fcf)
openssl-machine pushed a commit that referenced this pull request Oct 17, 2025
Signed-off-by: Joshua Rogers <[email protected]>

Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Saša Nedvědický <[email protected]>
(Merged from #28816)

(cherry picked from commit f7c0fcf)
openssl-machine pushed a commit that referenced this pull request Oct 17, 2025
Signed-off-by: Joshua Rogers <[email protected]>

Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Saša Nedvědický <[email protected]>
(Merged from #28816)

(cherry picked from commit f7c0fcf)
@t8m
Copy link
Member

t8m commented Oct 17, 2025

Merged to all the active branches. Thank you for your contribution.

@t8m t8m closed this Oct 17, 2025
MegaManSec added a commit to MegaManSec/openssl that referenced this pull request Nov 11, 2025
Signed-off-by: Joshua Rogers <[email protected]>

Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Saša Nedvědický <[email protected]>
(Merged from openssl#28816)
bernd-edlinger pushed a commit to bernd-edlinger/openssl that referenced this pull request Dec 21, 2025
Signed-off-by: Joshua Rogers <[email protected]>

Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Saša Nedvědický <[email protected]>
(Merged from openssl#28816)

(cherry picked from commit f7c0fcf)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Applies to master branch branch: 3.0 Applies to openssl-3.0 branch branch: 3.2 Applies to openssl-3.2 (EOL) branch: 3.3 Applies to openssl-3.3 branch: 3.4 Applies to openssl-3.4 branch: 3.5 Applies to openssl-3.5 branch: 3.6 Applies to openssl-3.6 tests: exempted The PR is exempt from requirements for testing triaged: bug The issue/pr is/fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants