Skip to content

Comments

ssl_do_config() name argument can be NULL#29306

Closed
Sashan wants to merge 1 commit intoopenssl:masterfrom
Sashan:CID.1675312
Closed

ssl_do_config() name argument can be NULL#29306
Sashan wants to merge 1 commit intoopenssl:masterfrom
Sashan:CID.1675312

Conversation

@Sashan
Copy link
Contributor

@Sashan Sashan commented Dec 4, 2025

Regression intorduced by #0c2a196af8ab9c12da27

Fixes Coverity issue 1675312

Regression intorduced by #0c2a196af8ab9c12da27

Fixes Coverity issue 1675312
@t8m t8m 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 tests: exempted The PR is exempt from requirements for testing labels Dec 4, 2025
@n13l
Copy link
Contributor

n13l commented Dec 4, 2025

LGTM. Good catch. Thank you.

@t8m t8m linked an issue Dec 4, 2025 that may be closed by this pull request
@npajkovsky npajkovsky 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 Dec 4, 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 Dec 5, 2025
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

@t8m
Copy link
Member

t8m commented Dec 5, 2025

Merged to the master branch with tweaked commit message. Thank you.

@t8m t8m closed this Dec 5, 2025
openssl-machine pushed a commit that referenced this pull request Dec 5, 2025
Fixes 0c2a196

Fixes Coverity issue 1675312

Reviewed-by: Nikola Pajkovsky <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #29306)
cxx194832 pushed a commit to cxx194832/openssl that referenced this pull request Dec 12, 2025
Fixes 0c2a196

Fixes Coverity issue 1675312

Reviewed-by: Nikola Pajkovsky <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from openssl#29306)
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 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.

Fix coverity 1675312

5 participants