Skip to content

Fix AES-CTR large chunk handling#309

Open
harrshita123 wants to merge 1 commit into
google:masterfrom
harrshita123:fix/aes-ctr-large-chunks
Open

Fix AES-CTR large chunk handling#309
harrshita123 wants to merge 1 commit into
google:masterfrom
harrshita123:fix/aes-ctr-large-chunks

Conversation

@harrshita123

Copy link
Copy Markdown
Contributor

Fixes #307.

Summary

  • account for bytes already consumed when selecting the next AES-CTR input chunk
  • add regression coverage for a 4097-byte input
  • verify byte encryption/decryption round-trips and matches streaming encryption

Testing

  • dart format --output none --set-exit-if-changed lib/src/impl_ffi/impl_ffi.aesctr.dart test/aes_ctr_counter_wrap_test.dart
  • dart analyze --fatal-warnings .
  • dart test -p vm test/aes_ctr_counter_wrap_test.dart
  • dart test -p chrome test/aes_ctr_counter_wrap_test.dart
  • dart test -p vm

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.

bug: native AES-CTR corrupts stream chunks larger than 4096 bytes

1 participant