[Pki-devel] [pki-devel][PATCH] 0016-Implement-Symmetric-Key-Changeover-Feature.patch

Christina Fu cfu at redhat.com
Sat Jul 26 01:51:54 UTC 2014


ACK.

However, for next patch:
* I think as long as they still apply, you should put the original 
comments (or the improvement of which) in the same place.
For instance, the original putKeys() puts the sample and layout of the 
key data at top of the function.  Your putKyes() has no comments at all.
Come to think of it, I might have missed this (lack of comments) in my 
previous reviews.  If you spot them, please put them in the future patchs.

Christina


On 07/25/2014 11:37 AM, John Magne wrote:
> [PATCH] Implement Symmetric Key Changeover Feature
>
> 1. Created code to actually change the key over to a new key set.
> 2. Made calls to the code from the format and enrollment operations,
> the Pin Reset processor will do so when completed.
> 3. Tested with real card. Was able to change to key set #2 and then back to stock
>
> Ticket #878.
>
>
>
>
> _______________________________________________
> Pki-devel mailing list
> Pki-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/pki-devel

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/pki-devel/attachments/20140725/649bf35e/attachment.htm>


More information about the Pki-devel mailing list