Skip to main content

Last Call Review of draft-ietf-lamps-cms-sha3-hash-03

Request Review of draft-ietf-lamps-cms-sha3-hash
Requested revision No specific revision (document currently at 04)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2024-05-03
Requested 2024-04-19
Authors Russ Housley
I-D last updated 2024-04-27
Completed reviews Secdir Last Call review of -03 by Wes Hardaker (diff)
Genart Last Call review of -03 by Dale R. Worley (diff)
Assignment Reviewer Dale R. Worley
State Completed
Request Last Call review on draft-ietf-lamps-cms-sha3-hash by General Area Review Team (Gen-ART) Assigned
Posted at
Reviewed revision 03 (document currently at 04)
Result Ready w/nits
Completed 2024-04-27
I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at


Document:  draft-ietf-lamps-cms-sha3-hash-03
Reviewer:  Dale R. Worley
Review Date:  2024-04-27
IETF LC End Date:  2024-05-03
IESG Telechat date:  [not known]


    This draft is basically ready for publication, but has a nit that
    should be fixed before publication.

I assume that the ASN.1 module has been reviewed by someone with the
necessary expertise.  (I do not have that expertise.)

Nits/editorial comments:

5.2.  KMAC128-KDF and KMAC256-KDF

The following is probably clearer if one is familiar with these
algorithms.  The text states

   The parameters are:

and follows with a list of four parameter values.  But later it says

   When the id-kmac128 or id-kmac256 is used as part of an algorithm
   identifier, the parameters field MUST be absent if no customization
   label is used for S.  If any other value is used for S, then
   parameters field MUST be present and contain the value of S, encoded
   as Customization.

Some differentiation should be made between the two senses of
"parameters".  In particular, it would help to state here where the K,
X, and L "parameters" are put, since they aren't put in the
"parameters field".

Also, the phrase "if no customization label is used for S" is not
quite correct, as it implies that something else could be "used for
S".  I think the correct wording is "if there is no customization
label S", which reflects that S has been stated to be "optional".