[PATCH] scd: support any curves defined by libgcrypt.
Bertrand Jacquin
bertrand at jacquin.bzh
Wed Aug 19 00:15:45 CEST 2015
HI,
On 17/08/2015 05:47, NIIBE Yutaka wrote:
> Hello,
>
> Thank you for the detailed report.
>
> On 08/17/2015 09:10 AM, Bertrand Jacquin wrote:
>> This patch introduce as issue with OpenGPG Card FTS-01. I've run a
>> git bisect to be able to point that particular commit.
>>
>> I have a FTS-01 OpenGPG card with a EdDSA sign key and a EdDSA auth
>> key and using gpg-agent as a ssh agent. Since that particular
>> commit, info given by gpg-agent to ssh-add are not conform for ssh:
>>
>> Is there a need to use a particular libgcrypt version ?
>
> No, it's not required for EdDSA. If you use encryption (ECDH with
> Curve25519), you need to use the development version of libgcrypt,
> though.
>
> I also noticed the failure on my side, and I posted:
>
> SSH with ed25519:
>
> https://lists.gnupg.org/pipermail/gnupg-devel/2015-August/030224.html
Thanks for having taking care of this. I can easily wait for that to be
properly fixed. Thanks !
> If I understand correctly, the problem is in the code of general EdDSA
> with SSH (I mean, including the private key in a host, as well as the
> one in smartcard). My change for scdaemon unveiled this issue.
>
> EdDSA with FST-01 just worked before, because of the public key
> representation from scdaemon didn't include the prefix of 0x40.
>
> My change of scdaemon for the support of more curves "fix"-ed to be
> proper representation between scdaemon<->gpg-agent and it now include
> the prefix of 0x40.
>
> I think that correct fix should be to the code in general EdDSA with
> SSH (not reverting a part of my change of scdaemon).
>
> The change in the post of mine works, but it's not good fix. This is
> basically to point out the issue. We need to confirm that gpg-agent
> handles EdDSA key with the representation of the prefix 0x40, and
> it is correct for its SSH handling to remove the prefix.
>
> Sorry, for your inconvenience.
--
Bertrand
More information about the Gnupg-devel
mailing list