Skip to content

script: Move more steps of AES algorithms to aes_common module#42323

Merged
TimvdLippe merged 1 commit intoservo:mainfrom
kkoyung:aes-move-more
Feb 4, 2026
Merged

script: Move more steps of AES algorithms to aes_common module#42323
TimvdLippe merged 1 commit intoservo:mainfrom
kkoyung:aes-move-more

Conversation

@kkoyung
Copy link
Copy Markdown
Member

@kkoyung kkoyung commented Feb 4, 2026

Move some common/similar steps of the import key operations of AES-CTR, AES-CBC, AES-GCM, AES-KW and AES-OCB to the shared module aes_common.

Some comments are also added to the shared module aes_common to explain the small difference in the specification of AES-OCB operations.

Testing: Refactoring. Existing tests suffice.

Move some common/similar steps of the import key operations of AES-CTR,
AES-CBC, AES-GCM, AES-KW and AES-OCB to the shared module `aes_common`.

Some comments are also added to the shared module `aes_common` to
explain the small difference in the specification of AES-OCB operations.

Testing: Refactoring. Existing tests suffice.

Signed-off-by: Kingsley Yung <[email protected]>
@kkoyung kkoyung requested a review from gterzian as a code owner February 4, 2026 05:13
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Feb 4, 2026
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Feb 4, 2026
@TimvdLippe TimvdLippe added this pull request to the merge queue Feb 4, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Feb 4, 2026
Merged via the queue into servo:main with commit db95883 Feb 4, 2026
32 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Feb 4, 2026
@kkoyung kkoyung deleted the aes-move-more branch March 7, 2026 03:11
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