Skip to content

EAX properly handle taglen boundaries#778

Open
karel-m wants to merge 1 commit into
developfrom
pr/eax-taglen
Open

EAX properly handle taglen boundaries#778
karel-m wants to merge 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
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.

1 participant