|
From: vmware <jia...@12...> - 2015-04-09 22:46:41
|
HI, Ludovic It’s my honour to be volunteer of the team. Is there any rules about how we run please let me know. thanks jianming On Apr 8, 2015, at 12:05 AM, Ludovic Rousseau <lud...@gm...> wrote: > 2015-04-03 15:50 GMT+02:00 Dirk-Willem van Gulik <di...@we...>: >> >> On 03 Apr 2015, at 15:06, Ludovic Rousseau <lud...@gm...> >> wrote: >> >> As it is not uncommon to do roll-over smartcards; which already have the >> ‘new’ cert or still have the expired cert. >> >> >> I changed the "break;" in "continue;" in >> https://github.com/OpenSC/pam_pkcs11/commit/4ef003ac43405f6391bf965a043f9fe4c4704f1d >> >> >> I think that is not quite enough - as those breaks where just breaking out >> of the switch; not out of the encapsulating loop. > > Exact. I reverted the patch. > >> Secondly - when NSS is in use - there is the issue that one cannot >> distinguish between normal invalid cert errors and nefarious/buggy errors >> within NSS/OpenSSL. >> >> So I guess at first oder we need something like: >> >> https://github.com/dirkx/pam_pkcs11/commit/f6a0b89d691b560ea546cab1bd74bc68c2dc1df5 >> >> and at second order - we could consider pulling line 670 onwards into the >> loop as well. To really try them one by one. > > Please create a Pull Request if you consider your code finished. > > I can also create a team of maintainers for pam_pkcs11 and add > volunteers to the team. Do you volunteer? > > Bye > > -- > Dr. Ludovic Rousseau |