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

Lint for CABF SMIME 7.1.4.2.h - If present, the subject:emailAddress SHALL contain a single Mailbox Address #753

Open
christopher-henderson opened this issue Oct 15, 2023 · 2 comments

Comments

@christopher-henderson
Copy link
Member

No description provided.

@CBonnell
Copy link
Contributor

The associated pull request is closed, so I'll comment here. The full text of 7.1.4.2.2 (h) says:

Certificate Field: subject:emailAddress (1.2.840.113549.1.9.1)
Contents: If present, the subject:emailAddress SHALL contain a single Mailbox Address as verified under Section 3.2.2.

Given that 7.1.4.2.2 is entitled "Subject distinguished name fields" and the specified OID "1.2.840.113549.1.9.1" denotes the PKCS #9 emailAddress attribute, the scope of the requirement is limited only to that single attribute. This passage is providing guidance that only a single Mailbox Address can be contained in the attribute value; it is not acceptable to include multiple Mailbox Addresses in a single attribute by delimiting them with a character (such as a space, semicolon, etc.).

Much in the same way that the TLS BRs do not prescribe an upper bound on the number of domain names/IP addresses allowed in a TLS certificate, the SMIME BRs do not prescribe an upper bound on the number of email addresses allowed in a SMIME certificate.

@cardonator
Copy link
Contributor

That makes sense, in this context the check should be that the contents of each element of the EmailAddresses array contain a single email address with no additional characters or email addresses.

cardonator added a commit to cardonator/zlint that referenced this issue Jan 30, 2024
…ultiple email address subject fields, but dsisallow multiple values in a single email address field

fixes a comment on zmap#753
zakird pushed a commit that referenced this issue Feb 9, 2024
…t:emailAddress (#792)

* fix bug in the email address checking in the smime package to allow multiple email address subject fields, but dsisallow multiple values in a single email address field

fixes a comment on #753

* fix typo
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

No branches or pull requests

3 participants