On 05/31/2013 08:20 PM, John Magne wrote:
Please review the additional patch included: subject.diff

This is a fix to the CA for this same issue that will prevent enrollments with illegal attributes contained within the Subject Name for an incoming request.

----- Forwarded Message -----
From: "John Magne" <jmagne@redhat.com>
To: "pki-devel" <pki-devel@redhat.com>
Sent: Wednesday, May 29, 2013 10:41:18 AM
Subject: Review Request  Bug 963073 - rhcs81 tps crash for CN over than 64 bytes


Please review attached patch. Fix solves a crash when a certificate to be displayed contains too many characters in one of its fields.
_______________________________________________ Pki-devel mailing list Pki-devel@redhat.com https://www.redhat.com/mailman/listinfo/pki-devel

Looks good overall. 
One thing to consider is to check for all fields before throwing an exception listing all violated components. 

Otherwise ack.

Christina