> On 7 Jul 2023, at 18:35, Mark Johnston <ma...@freebsd.org> wrote:
>
> On Wed, Jul 05, 2023 at 11:56:42PM +0200, Kristof Provost wrote:
>>> On 24 Jun 2023, at 1:19, Ed Maste wrote:
>>> The branch main has been updated by emaste:
>>>
>>> URL:
>>> https://cgit.FreeBSD.org/src/commit/?id=b077aed33b7b6aefca7b17ddb250cf521f938613
>>>
>>> commit b077aed33b7b6aefca7b17ddb250cf521f938613
>>> Merge: b08ee10c0646 b84c4564effd
>>> Author: Pierre Pronchery <pie...@freebsdfoundation.org>
>>> AuthorDate: 2023-06-23 22:53:35 +0000
>>> Commit: Ed Maste <ema...@freebsd.org>
>>> CommitDate: 2023-06-23 22:53:36 +0000
>>>
>>> Merge OpenSSL 3.0.9
>>>
>>
>> It looks like we missed adding a file.
>> Security/opensc doesn’t build any more:
>> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=270076
>>
>> It fails to find d2i_KeyParams when linking. The opensc code does this:
>>
>> #if OPENSSL_VERSION_NUMBER < 0x30000000L
>> if (!d2i_ECParameters(&ec, &a, (long)len))
>> util_fatal("cannot parse
>> EC_PARAMS");
>> EVP_PKEY_assign_EC_KEY(pkey, ec);
>> #else
>> if (!d2i_KeyParams(EVP_PKEY_EC, &pkey, &a,
>> len))
>> util_fatal("cannot parse
>> EC_PARAMS");
>> #endif
>>
>> d2i_KeyParams() appears to be new on openssl 3. It’s defined in d2i_param.c,
>> which we don’t build. I’ve tested with this patch, and that appears to fix
>> things:
>
> Hi Kristof,
>
> Would you mind posting the patch on phabricator? I can take a closer
> look in the next day, and Pierre might be available to look as well.
Sure, but I might not be able to do that until Sunday afternoon.
>
>> diff --git a/secure/lib/libcrypto/Makefile b/secure/lib/libcrypto/Makefile
>> index 28258e796984..ef5652e8c27c 100644
>> --- a/secure/lib/libcrypto/Makefile
>> +++ b/secure/lib/libcrypto/Makefile
>> @@ -74,7 +74,7 @@ SRCS+= n_pkey.c nsseq.c p5_pbe.c p5_pbev2.c
>> p5_scrypt.c p8_pkey.c
>> SRCS+= t_bitst.c t_pkey.c t_spki.c tasn_dec.c tasn_enc.c tasn_fre.c
>> SRCS+= tasn_new.c tasn_prn.c tasn_scn.c tasn_typ.c tasn_utl.c x_algor.c
>> SRCS+= x_bignum.c x_info.c x_int64.c x_long.c x_pkey.c x_sig.c x_spki.c
>> -SRCS+= x_val.c
>> +SRCS+= x_val.c d2i_param.c
>>
>> # async
>> SRCS+= async.c async_err.c async_posix.c async_wait.c
>> diff --git a/secure/lib/libcrypto/Version.map
>> b/secure/lib/libcrypto/Version.map
>> index 421819324961..74d0b8b3cef1 100644
>> --- a/secure/lib/libcrypto/Version.map
>> +++ b/secure/lib/libcrypto/Version.map
>> @@ -3564,6 +3564,8 @@ OPENSSL_1_1_0 {
>> d2i_IPAddressOrRange;
>> d2i_IPAddressRange;
>> d2i_ISSUING_DIST_POINT;
>> + d2i_KeyParams;
>> + d2i_KeyParams_bio;
>
> Based on your analysis I think this should go into the OPENSSL_3_0_9
> namespace?
>
I have no idea. I’ll try to dig a bit, but we’re pretty far outside my comfort
zone here.
Best regards,
Kristof
>
>
