[Pki-devel] [pki-devel][PATCH] 0016-Implement-Symmetric-Key-Changeover-Feature.patch
John Magne
jmagne at redhat.com
Sat Jul 26 01:56:58 UTC 2014
Based on cfu's ack,
pushed to master.
Will deal with missing comment blocks conveniently in next submission.
----- Original Message -----
> From: "Christina Fu" <cfu at redhat.com>
> To: pki-devel at redhat.com
> Sent: Friday, July 25, 2014 6:51:54 PM
> Subject: Re: [Pki-devel] [pki-devel][PATCH] 0016-Implement-Symmetric-Key-Changeover-Feature.patch
>
> 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
>
>
> _______________________________________________
> Pki-devel mailing list
> Pki-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/pki-devel
More information about the Pki-devel
mailing list