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

Add Cipher_Mode::ideal_granularity #3168

Merged
merged 5 commits into from
Jan 12, 2023
Merged

Add Cipher_Mode::ideal_granularity #3168

merged 5 commits into from
Jan 12, 2023

Conversation

randombit
Copy link
Owner

This reflects the size that would be best for performance, while update_granularity is modified to return the true minimum value.

GH #3163

This reflects the size that would be best for performance, while
update_granularity is modified to return the true minimum value.

GH #3163
src/lib/ffi/ffi_cipher.cpp Outdated Show resolved Hide resolved
src/tests/test_aead.cpp Outdated Show resolved Hide resolved
src/tests/test_modes.cpp Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented Jan 11, 2023

Codecov Report

Base: 88.04% // Head: 88.04% // Decreases project coverage by -0.00% ⚠️

Coverage data is based on head (eff936f) compared to base (feb75cc).
Patch coverage: 92.10% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3168      +/-   ##
==========================================
- Coverage   88.04%   88.04%   -0.01%     
==========================================
  Files         604      605       +1     
  Lines       67226    67347     +121     
  Branches     6735     6739       +4     
==========================================
+ Hits        59192    59298     +106     
- Misses       5212     5222      +10     
- Partials     2822     2827       +5     
Impacted Files Coverage Δ
src/lib/tls/tls12/tls_cbc/tls_cbc.cpp 84.26% <0.00%> (-0.79%) ⬇️
src/lib/ffi/ffi_cipher.cpp 89.21% <76.47%> (-2.00%) ⬇️
src/lib/filters/cipher_filter.cpp 92.68% <100.00%> (ø)
src/lib/modes/aead/ccm/ccm.cpp 90.07% <100.00%> (+0.23%) ⬆️
...b/modes/aead/chacha20poly1305/chacha20poly1305.cpp 94.18% <100.00%> (+0.35%) ⬆️
src/lib/modes/aead/eax/eax.cpp 94.04% <100.00%> (+0.22%) ⬆️
src/lib/modes/aead/gcm/gcm.cpp 92.59% <100.00%> (+2.46%) ⬆️
src/lib/modes/aead/ocb/ocb.cpp 98.37% <100.00%> (+0.44%) ⬆️
src/lib/modes/aead/siv/siv.cpp 91.75% <100.00%> (+0.26%) ⬆️
src/lib/modes/cbc/cbc.cpp 92.66% <100.00%> (+0.09%) ⬆️
... and 23 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@randombit randombit merged commit b783fa1 into master Jan 12, 2023
randombit added a commit that referenced this pull request Jan 13, 2023
Previously update_granularity == 1 was an (undocumented and not
reliable) method of detecting if an AEAD mode required presenting the
entire message before processing could occur.  Since #3168 this
assumption is no longer valid.

Add a method which directly indicates if the cipher mode has this
nature.
@randombit randombit deleted the jack/ideal-granularity branch January 14, 2023 13:36
randombit added a commit that referenced this pull request Jan 15, 2023
That PR accidentally disabled decrypting multiple blocks in parallel.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants