On Mon, 2011-02-28 at 15:33 +0100, Andre Zepezauer wrote:
> Hello Martin,
> 
> I would like to commit the attached patch. Any objections?

Committed in r5222.

> On Thu, 2011-02-03 at 14:36 +0200, Martin Paljak wrote:
> > Hello,
> > 
> > On Thu, Jan 27, 2011 at 20:08, Andre Zepezauer
> > <andre.zepeza...@student.uni-halle.de> wrote:
> > > Hello Martin,
> > >
> > > some comments on r5124:
> > >
> > > 1. The values of pin_info->reference and prkey_info->key_reference
> > > shouldn't be compared because:
> > >
> > > * pin_info->reference is used as P2 parameter in VERIFY command
> > > * prkey_info->key_reference is used in MSE SET tag 0x84
> > 
> > OK, I see your point.
> > Looking at your patch: could it be extracted into a small lookup
> > function like the current one that is used? such a small lookup
> > function with a small doxygen doc would look really nice.
> > 
> > I see it has been working up to because of a coincidence...
> _______________________________________________
> opensc-devel mailing list
> opensc-devel@lists.opensc-project.org
> http://www.opensc-project.org/mailman/listinfo/opensc-devel

_______________________________________________
opensc-devel mailing list
opensc-devel@lists.opensc-project.org
http://www.opensc-project.org/mailman/listinfo/opensc-devel

Reply via email to