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

openssl speed fails to count for aead decryption #24404

Open
niyazmurshed opened this issue May 15, 2024 · 3 comments
Open

openssl speed fails to count for aead decryption #24404

niyazmurshed opened this issue May 15, 2024 · 3 comments
Labels
branch: master Merge to master branch branch: 3.2 Merge to openssl-3.2 branch: 3.3 Merge to openssl-3.3 help wanted severity: regression The issue/pr is a regression from previous released version triaged: bug The issue/pr is/fixes a bug

Comments

@niyazmurshed
Copy link

Hi,
I have recently started using version (OpenSSL 3.3.0-dev) where I am looking at aead encryption and decryption.
It seems that something has been broken in decryption. Has something changed from previous version?
I don't see any change in speed.c that might have caused this.

root# openssl speed -evp aes-128-gcm -bytes 1424 --aead -decrypt 
Doing AES-128-GCM for 3s on 1424 size blocks: 3878862 AES-128-GCM's in 3.00s
version: 3.0.2
built on: Fri Feb 16 08:51:30 2024 UTC
options: bn(64,64)
compiler: gcc -fPIC -pthread -Wa,--noexecstack -Wall -Wa,--noexecstack -g -O2 -ffile-prefix-map=/build/openssl-luvhhy/openssl-3.0.2=. -flto=auto -ffat-lto-objects -flto=auto -ffat-lto-objects -fstack-protector-strong -Wformat -Werror=format-security -DOPENSSL_TLS_SECURITY_LEVEL=2 -DOPENSSL_USE_NODELETE -DOPENSSL_PIC -DOPENSSL_BUILDING_OPENSSL -DNDEBUG -Wdate-time -D_FORTIFY_SOURCE=2
CPUINFO: OPENSSL_armcap=0xbf
The 'numbers' are in 1000s of bytes per second processed.
type           1424 bytes
**AES-128-GCM    1841166.50k**



root# openssl speed -evp aes-128-gcm -bytes 1400 --aead -decrypt 
Doing AES-128-GCM ops for 3s on 1400 size blocks: Running aead 
0 AES-128-GCM ops in 2.99s
version: 3.3.0-dev
built on: Mon Apr  8 17:26:59 2024 UTC
options: bn(64,64)
compiler: gcc -fPIC -pthread -Wa,--noexecstack -Wall -O3 -DOPENSSL_USE_NODELETE -DOPENSSL_PIC -DOPENSSL_BUILDING_OPENSSL -DZLIB -DNDEBUG
CPUINFO: OPENSSL_armcap=0xbd
The 'numbers' are in 1000s of bytes per second processed.
type           1400 bytes
**AES-128-GCM          0.00**
@niyazmurshed niyazmurshed added the issue: bug report The issue was opened to report a bug label May 15, 2024
@t8m
Copy link
Member

t8m commented May 15, 2024

This is regression from 98283a6

@t8m t8m added branch: master Merge to master branch help wanted triaged: bug The issue/pr is/fixes a bug severity: regression The issue/pr is a regression from previous released version branch: 3.2 Merge to openssl-3.2 branch: 3.3 Merge to openssl-3.3 and removed issue: bug report The issue was opened to report a bug labels May 15, 2024
@t8m
Copy link
Member

t8m commented May 15, 2024

@beldmit Unfortunately when decrypting we cannot really check the result of the final calls. We would have to use real data and not something fictious.

@t8m t8m changed the title openssl aead decryption test openssl speed fails to count for aead decryption May 15, 2024
@beldmit
Copy link
Member

beldmit commented May 15, 2024

@t8m we can expect failure instead of success in case of decrypt, I think. Or honestly do encryption first...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch: master Merge to master branch branch: 3.2 Merge to openssl-3.2 branch: 3.3 Merge to openssl-3.3 help wanted severity: regression The issue/pr is a regression from previous released version triaged: bug The issue/pr is/fixes a bug
Projects
Status: New
Development

No branches or pull requests

3 participants