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

[pentest] Add AES-GCM PTX&AAD FvsR #27

Merged
merged 1 commit into from
Feb 23, 2025

Conversation

nasahlpa
Copy link
Collaborator

Previously, only the key and iv were in the FvsR set. With this commit, the user can decide:

  • Key&IV FvsR: key&IV in FvsR set, AAD&PTX static
  • PTX&AAD FvsR: PTX&AAD in FvsR set, key/IV static

@nasahlpa nasahlpa requested a review from vogelpi February 20, 2025 10:43
Previously, only the key and iv were in the FvsR set. With this
commit, the user can decide:
- Key&IV FvsR: key&IV in FvsR set, AAD&PTX static
- PTX&AAD FvsR: PTX&AAD in FvsR set, key/IV static

Signed-off-by: Pascal Nasahl <[email protected]>
Copy link
Owner

@vogelpi vogelpi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @nasahlpa !

@nasahlpa nasahlpa merged commit ec0d795 into vogelpi:aes-gcm-review Feb 23, 2025
7 of 9 checks passed
@nasahlpa nasahlpa deleted the fvsr_input branch February 23, 2025 15:32
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.

2 participants