|
From: jianming <jia...@12...> - 2015-04-09 22:50:45
|
sorry, change the email sender from vmware to jianming due to email config error. On Apr 10, 2015, at 6:46 AM, vmware <jia...@12...> wrote: > 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 > |