Skip to content

EAX properly handle taglen boundaries#778

Merged
sjaeckel merged 1 commit into
developfrom
pr/eax-taglen
Jul 21, 2026
Merged

EAX properly handle taglen boundaries#778
sjaeckel merged 1 commit into
developfrom
pr/eax-taglen

Conversation

@karel-m

@karel-m karel-m commented Jul 18, 2026

Copy link
Copy Markdown
Member

This issue was initially reported privately by @stigtsp as a potential vulnerability in CryptX (perl bindings for libtomcrypt).

The reported problem was that EAX accepts zero-length authentication tags.

I therefore checked the EAX specification:

It states:

Any tag length T [0..n] should be possible, to allow each user to select how much security she wants from the integrity guarantees and how many bits she has to pay for this.

A zero-length tag is therefore valid. It provides no authentication, but that is the caller's choice.

However, I found that we were not correctly checking the upper bound of the tag length.

This patch:

  • correctly checks the upper bound of the EAX tag length
  • updates the documentation to clarify that a zero-length EAX tag is valid

@karel-m
karel-m requested a review from sjaeckel July 18, 2026 21:18
@sjaeckel
sjaeckel merged commit b7b04ff into develop Jul 21, 2026
21 of 202 checks passed
@sjaeckel
sjaeckel deleted the pr/eax-taglen branch July 21, 2026 08:59
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