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.

Sign up for GitHub

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

Jump to bottom

Use of direct cipher and digest commands crashes openssl(1) in 3.0.0 and master #16713

Closed
vdukhovni opened this issue Sep 29, 2021 · 0 comments
Closed

Use of direct cipher and digest commands crashes openssl(1) in 3.0.0 and master #16713

vdukhovni opened this issue Sep 29, 2021 · 0 comments
Labels
branch: master Merge to master branch branch: 3.0 Merge to openssl-3.0 branch triaged: bug The issue/pr is/fixes a bug

Comments

@vdukhovni
Copy link

vdukhovni commented Sep 29, 2021 •
edited

In 3.0.0 the openssl(1) <cipher|digest> commands crash because the function structure is not fully initialised, and inadvertently enters the deprecation warning code as a result.
Example (reported to openssl-users by "Steffen Nurpmeso"):

openssl chacha20

(really any cipher or digest name).
@vdukhovni vdukhovni added the issue: bug report The issue was opened to report a bug label Sep 29, 2021
@vdukhovni vdukhovni mentioned this issue Sep 29, 2021
Closed
vdukhovni pushed a commit to vdukhovni/openssl that referenced this issue Sep 30, 2021
Fully initialise cipher/digest app handles
7cb972f 
This avoids a crash in e.g. `openssl chacha20` as reported by
Steffen Nurpmeso on openssl-users.

Resolves: openssl#16713
@vdukhovni vdukhovni changed the title Use of deprecated cipher and digest commands crashes openssl(1) in 3.0.0 and master Use of direct cipher and digest commands crashes openssl(1) in 3.0.0 and master Sep 30, 2021
@t8m t8m added branch: 3.0 Merge to openssl-3.0 branch branch: master Merge to master branch triaged: bug The issue/pr is/fixes a bug and removed issue: bug report The issue was opened to report a bug labels Sep 30, 2021
openssl-machine pushed a commit that referenced this issue Oct 1, 2021
@paulidale
Fully initialise cipher/digest app handles
ff8f2d0 
This avoids a crash in e.g. `openssl chacha20` as reported by
Steffen Nurpmeso on openssl-users.

Resolves: #16713

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from #16714)

(cherry picked from commit a8cc0ef)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
branch: master Merge to master branch branch: 3.0 Merge to openssl-3.0 branch triaged: bug The issue/pr is/fixes a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Initialise cipher/digest deprecation structures better vdukhovni/openssl
2 participants
@vdukhovni @t8m