zmap / zlint

X.509 Certificate Linter focused on Web PKI standards and requirements.
https://zmap.io
Apache License 2.0
361 stars 110 forks source link

Bug in lint for BR 7.1.4.2.2a mailbox-validated #844

Closed robstradling closed 6 months ago

robstradling commented 6 months ago

The lint added in PR #806 (https://github.com/zmap/zlint/commit/7c899eaaaa534b10489f457ffbea808235d4fc71) incorrectly handles the case where Subject:commonName is not present in a Mailbox Validation S/MIME certificate.

When there is no Subject:commonName in a certificate, Subject.CommonName in the z509 Certificate object is an empty string. The bug is that the e_commonname_mailbox_validated lint is passing this empty string to util.IsMailboxAddress instead of ignoring it, resulting in a false positive error.

This bug blocks Sectigo from upgrading to ZLint v3.6.2.