Re: [Technical Errata Reported] RFC2634 (6562)
Russ Housley <[email protected]> Wed, 28 Apr 2021 15:06:32 -0400
| Newsgroups | gmane.ietf.smime |
|---|---|
| Message-ID | <[email protected]> |
Roman and Ben: This was discussed on the mail list, and people agree that the proposed text adds clarity. I think that the MUST in the first sentence was implied, but others think otherwise. I recommend approving this one. Russ > On Apr 28, 2021, at 2:07 PM, RFC Errata System <[email protected]> wrote: > > The following errata report has been submitted for RFC2634, > "Enhanced Security Services for S/MIME". > > -------------------------------------- > You may review the report below and at: > https://www.rfc-editor.org/errata/eid6562 > > -------------------------------------- > Type: Technical > Reported by: David von Oheimb <[email protected]> > > Section: 5.4 > > Original Text > ------------- > The first certificate identified in the sequence of certificate > identifiers MUST be the certificate used to verify the signature. The > encoding of the ESSCertID for this certificate SHOULD include the > issuerSerial field. If other constraints ensure that > issuerAndSerialNumber will be present in the SignerInfo, the > issuerSerial field MAY be omitted. The certificate identified is used > during the signature verification process. If the hash of the > certificate does not match the certificate used to verify the > signature, the signature MUST be considered invalid. > > If more than one certificate is present in the sequence of > ESSCertIDs, the certificates after the first one limit the set of > authorization certificates that are used during signature validation. > > > Corrected Text > -------------- > The sequence of certificate identifiers MUST contain at least one element. > The first certificate identified MUST be the certificate used to verify the signature. > The encoding of the ESSCertID for this certificate SHOULD include the > issuerSerial field. If other constraints ensure that > issuerAndSerialNumber will be present in the SignerInfo, the > issuerSerial field MAY be omitted. The certificate identified is used > during the signature verification process. If the hash of the > certificate does not match the certificate used to verify the > signature, the signature MUST be considered invalid. > > If more than one certificate identifier is present in the sequence of ESSCertIDs, > all certificates referenced there MUST be part of the signature validation chain. > > > Notes > ----- > Some aspects of the 'certs' field of a SigningCertificate attribute: > > SigningCertificate ::= SEQUENCE { > certs SEQUENCE OF ESSCertID, > policies SEQUENCE OF PolicyInformation OPTIONAL > } > > described in the sentences quoted above are very vague. > This lead to major confusion and wrong implementations. > As meanwhile has been clarified, they should be re-phrased; > see suggested new version above. > > (One may further mandate/clarify that the certificate identifiers must be given in the same order > as they are expected in the validation chain, but I think this is not important because > the order should not play a critical role and will be determined by the validation chain anyway.) > > Instructions: > ------------- > This erratum is currently posted as "Reported". If necessary, please > use "Reply All" to discuss whether it should be verified or > rejected. When a decision is reached, the verifying party > can log in to change the status and edit the report, if necessary. > > -------------------------------------- > RFC2634 (draft-ietf-smime-ess-12) > -------------------------------------- > Title : Enhanced Security Services for S/MIME > Publication Date : June 1999 > Author(s) : P. Hoffman, Ed. > Category : PROPOSED STANDARD > Source : S/MIME Mail Security > Area : Security > Stream : IETF > Verifying Party : IESG > > _______________________________________________ > smime mailing list > [email protected] > https://www.ietf.org/mailman/listinfo/smime