Skip to main content

Last Call Review of draft-ietf-cose-aes-ctr-and-cbc-04
review-ietf-cose-aes-ctr-and-cbc-04-secdir-lc-migault-2023-05-16-00

Request Review of draft-ietf-cose-aes-ctr-and-cbc
Requested revision No specific revision (document currently at 06)
Type Last Call Review
Team Security Area Directorate (secdir)
Deadline 2023-05-05
Requested 2023-04-21
Authors Russ Housley , Hannes Tschofenig
I-D last updated 2023-05-16
Completed reviews Genart Last Call review of -04 by Vijay K. Gurbani (diff)
Secdir Last Call review of -04 by Daniel Migault (diff)
Assignment Reviewer Daniel Migault
State Completed
Request Last Call review on draft-ietf-cose-aes-ctr-and-cbc by Security Area Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/secdir/i1mBNVBu2721rTcvZ3wUrgRzqHc
Reviewed revision 04 (document currently at 06)
Result Ready
Completed 2023-05-16
review-ietf-cose-aes-ctr-and-cbc-04-secdir-lc-migault-2023-05-16-00
Reviewer: Daniel Migault
Review result: Ready

I have reviewed this document as part of the security directorate's ongoing
effort to review all IETF documents being processed by the IESG. These comments
were written primarily for the benefit of the security area directors. Document
editors and WG chairs should treat these comments just like any other

section 4 AES counter mode

In "AES encryption of (IV +1) mod 2^128" I am wondering if "mod 2^128" is
needed as I see the encryption returning a 128 bit block. That said we
understand why it is there, it is more that I am curious if there is any reason.

I am also wondering if we should mention the IV + i is called the counter block
as this is mentioned in section 8.

The following text sounded cryptic to me until I reached section 6. I suspect
that adding a reference to section 6 might be useful. The same comment applies
for CBC.

"""
Since AES-CTR cannot provide integrity protection for external
additional authenticated data, the decryptor MUST ensure that no
external additional authenticated data was supplied.
"""

section 4.2.  AES-CTR COSE Algorithm Identifiers

In the title “Algoritm” needs to be changed.

It is surprising to define a "Deprecated", but the note provides the rationale.
I am wondering if that rationale could be also mentioned in the IANA page -
this is just a suggestion.

section 5.  AES Cipher Block Chaining Mode

I believe that another reason for using integrity protection is the
vulnerability to padding oracle.