Page MenuHome GnuPG

libgcryptProject
ActivePublic

Recent Activity

Mon, Dec 4

werner triaged T6858: libgcrypt fails to be cross-compiled. as Normal priority.

You may better ask on gcrypt-devel at gnupg.org for help.

Mon, Dec 4, 4:57 PM · libgcrypt

Fri, Dec 1

ametzler1 created T6863: [patch] libgcrypt copyright header cleanup.
Fri, Dec 1, 6:21 PM · patch, libgcrypt, Bug Report

Thu, Nov 30

twaik created T6858: libgcrypt fails to be cross-compiled..
Thu, Nov 30, 2:04 PM · libgcrypt

Tue, Nov 28

fse added a comment to T6637: PQC for Libgcrypt.

And another question: in the GnuPG code on the master branch I saw that algorithm identifiers for ML-KEM with Ed25519 and Ed448 are already defined in the code base. Do I understand correctly that the maintainers prefer the inclusion of these two algorithms and not necessarily the inclusion of the ones based on ML-KEM with ECDH using NIST or Brainpool curves?

Tue, Nov 28, 1:21 PM · PQC, libgcrypt

Mon, Nov 27

fse added a comment to T6637: PQC for Libgcrypt.

We have addressed all comments regarding ML-KEM (Kyber) and KMAC. Currently I am working on the GnuPG integration of the the ML-KEM composites. For that purpose I will need a branch of libgcrypt with both ML-KEM and KMAC. I am not sure if you are considering to integrate the ML-KEM version already now before the final NIST standards are release. Some libraries do it, for instance Botan. Appropriate naming of the algorithms can ensure that there arises no confusion which version of the algorithm one is using.

Mon, Nov 27, 4:30 PM · PQC, libgcrypt

Thu, Nov 16

werner closed T6335: Release Libgcrypt 1.8.11 as Resolved.
Thu, Nov 16, 10:55 AM · libgcrypt, Release Info

Wed, Nov 15

gniibe closed T3264: Possible RSA improvement as Invalid.

RSA improvement is not that worth now.

Wed, Nov 15, 1:14 AM · libgcrypt
gniibe closed T3269: (Constant-time) modular reduction as Resolved.

OK. When we will need and do, I will open new one.

Wed, Nov 15, 1:12 AM · libgcrypt
gniibe closed T3269: (Constant-time) modular reduction, a subtask of T3264: Possible RSA improvement, as Resolved.
Wed, Nov 15, 1:12 AM · libgcrypt
gniibe closed T6539: The digest&sign/verify API with SHAKE-class digests does not work as Resolved.

The fix is in 1.10.3.

Wed, Nov 15, 1:02 AM · libgcrypt, FIPS, Bug Report
gniibe closed T6507: SCRYPT does not work in FIPS mode as Resolved.

Fix is in 1.10.3.

Wed, Nov 15, 12:54 AM · libgcrypt, FIPS, Bug Report

Tue, Nov 14

werner moved T3269: (Constant-time) modular reduction from For 1.10 to Backlog on the libgcrypt board.

@gniibe: This is a pretty old bug; given all the changes of the last year, should we close it now.

Tue, Nov 14, 1:21 PM · libgcrypt
werner closed T6747: sexp string including \0 as Resolved.
Tue, Nov 14, 1:18 PM · libgcrypt, Bug Report
werner closed T6217: sha3: wrong results for large inputs as Resolved.
Tue, Nov 14, 1:18 PM · libgcrypt, FIPS, Bug Report
werner closed T4873: Enable AES GCM in FIPS mode as Resolved.
Tue, Nov 14, 1:17 PM · FIPS, libgcrypt, Feature Request
werner closed T4873: Enable AES GCM in FIPS mode, a subtask of T5870: libgcrypt: AEAD API for FIPS 140 (in future), as Resolved.
Tue, Nov 14, 1:17 PM · Feature Request, FIPS, libgcrypt
werner moved T6747: sexp string including \0 from Backlog to For 1.10 on the libgcrypt board.
Tue, Nov 14, 1:15 PM · libgcrypt, Bug Report
werner moved T6217: sha3: wrong results for large inputs from Backlog to For 1.10 on the libgcrypt board.
Tue, Nov 14, 1:14 PM · libgcrypt, FIPS, Bug Report
werner closed T6817: Release Libgcrypt 1.10.3 as Resolved.
Tue, Nov 14, 1:13 PM · Release Info, libgcrypt
werner closed T5905: Release Libgcrypt 1.10.2 as Resolved.
Tue, Nov 14, 12:55 PM · Release Info, libgcrypt
werner created T6817: Release Libgcrypt 1.10.3.
Tue, Nov 14, 12:54 PM · Release Info, libgcrypt

Oct 31 2023

gniibe added a comment to T6637: PQC for Libgcrypt.

In master, when fixing padding issue, libgcrypt/src/const-time.h is just introduced.
I will replace your functions.

Oct 31 2023, 7:41 AM · PQC, libgcrypt

Oct 24 2023

fse added a comment to T6637: PQC for Libgcrypt.

Yes, int8_t/int16_t/int32_t/uint8_t/uint16_t/uint32_t should not be used. There is size-specific integer types defined in src/types.h which can be used instead (byte/u16/u32). This header does not yet have signed integer types, but those can be added (for example, s8/s16/s32).

Oct 24 2023, 1:34 PM · PQC, libgcrypt

Oct 23 2023

aheinecke added a comment to T6637: PQC for Libgcrypt.
In T6637#176910, @fse wrote:

OK, fine, however, in order to be able keep an overview of our tasks I would still keep track of them in our GitHub, where I can create a sub-issue from the list of tasks with one click. But we will post our comments and results here as well as far relevant for the purpose of documentation. I think most of the points Jussi raised are more or less clear to me anyway.

Oct 23 2023, 7:23 PM · PQC, libgcrypt
jukivili added a comment to T6637: PQC for Libgcrypt.

Yes, int8_t/int16_t/int32_t/uint8_t/uint16_t/uint32_t should not be used. There is size-specific integer types defined in src/types.h which can be used instead (byte/u16/u32). This header does not yet have signed integer types, but those can be added (for example, s8/s16/s32).

Oct 23 2023, 7:00 PM · PQC, libgcrypt
jukivili closed T6451: libgcrypt | gcry_cipher_setkey: 3DES-CBC key returns GPG_ERR_WEAK even with GCRYCTL_SET_ALLOW_WEAK_KEY as Resolved.
Oct 23 2023, 6:56 PM · Debian, libgcrypt, Bug Report

Oct 18 2023

fse added a comment to T6637: PQC for Libgcrypt.

@jukivilli I have addressed a number of your comments now. You find my comments inline.

Oct 18 2023, 1:33 PM · PQC, libgcrypt

Oct 17 2023

jukivili updated the task description for T4460: libgcrypt performance TODOs.
Oct 17 2023, 5:57 PM · libgcrypt

Oct 16 2023

fse added a comment to T6755: libgcrypt: KEM API.

Yes, apparently I confused uint8_t and unsigned char here because the former appears in Simon's comments. We also kept to the use of unsigned char* in our implementations (that is even part of the GNU coding guidelines if I remember correctly).

Oct 16 2023, 1:43 PM · PQC, libgcrypt
werner added a comment to T6755: libgcrypt: KEM API.

Actually we never use uint8_t* because that is c99 and very uncommon except for some MCU projects. Instead we use unsigned char *. The use of void* is often used because this allows to pass arbitrary types to a function without requiring ugly and error-prone casting at the caller site.

Oct 16 2023, 1:14 PM · PQC, libgcrypt
werner added a comment to T6637: PQC for Libgcrypt.

You don't need a library but just one object file.

Oct 16 2023, 12:57 PM · PQC, libgcrypt
fse added a comment to T6637: PQC for Libgcrypt.

OK, fine, however, in order to be able keep an overview of our tasks I would still keep track of them in our GitHub, where I can create a sub-issue from the list of tasks with one click. But we will post our comments and results here as well as far relevant for the purpose of documentation. I think most of the points Jussi raised are more or less clear to me anyway.

Oct 16 2023, 12:07 PM · PQC, libgcrypt
fse added a comment to T6755: libgcrypt: KEM API.

With respect to the function signatures, I see the following issues with the API you reference via the provided link:

Oct 16 2023, 12:01 PM · PQC, libgcrypt
werner added a comment to T6637: PQC for Libgcrypt.

@fse: Github is not an option here. We don't use it and thus everything relevant to Libgcrypt needs to be documented here and not at some external platform.

Oct 16 2023, 11:53 AM · PQC, libgcrypt
gniibe added a comment to T6755: libgcrypt: KEM API.

For length information, we can find that Simon's patch (let me call it v1) has length argument:
https://gitlab.com/jas/libgcrypt/-/commit/3af635afca052a9575912b257fe7518a58bfe810

Oct 16 2023, 10:24 AM · PQC, libgcrypt
fse added a comment to T6637: PQC for Libgcrypt.

Hi Jussi,

Oct 16 2023, 8:37 AM · PQC, libgcrypt

Oct 15 2023

jukivili added a comment to T6637: PQC for Libgcrypt.
  • There's many functions that use buffers on stack. Do those contain secrets? Should those buffers be wiped before returning from function (with wipememory())? For example, "mlkem_check_secret_key" has two buffers "shared_secret_1" and "shared_secret_2" which are not wiped.
  • mlkem.c: mlkem_check_secret_key: "memcmp" is used to compare shared secrets. Should this use constant time comparison instead?
  • mlkem-common.c: _gcry_mlkem_mlkem_shake256_rkprf:
    • _gcry_md_hash_buffers_extract can be used here instead of _gcry_md_open&write&extract&close.
  • mlkem-symmetric.c: _gcry_mlkem_shake256_prf:
    • _gcry_md_hash_buffers_extract can be used here instead of _gcry_md_open&write&extract&close. Temporary buffer usage can be avoided by passing input buffers through two IOV to _gcry_md_hash_buffers_extract.
Oct 15 2023, 5:08 PM · PQC, libgcrypt
jukivili added a comment to T6637: PQC for Libgcrypt.

Few comments on the patches.

Oct 15 2023, 4:38 PM · PQC, libgcrypt

Oct 11 2023

fse added a comment to T6755: libgcrypt: KEM API.

Our own internal function signatures is not necessarily a good refernce. The main objection to all what you list above is the lack of explicit length information. For each uint8_t* there should also be a size_t ...len in my opinion. Otherwise the API will be highly prone to memory access errors.

Oct 11 2023, 8:34 AM · PQC, libgcrypt
gniibe added a comment to T6755: libgcrypt: KEM API.

@fse Thank you for your comment (quick ! :-).

Oct 11 2023, 6:47 AM · PQC, libgcrypt

Oct 10 2023

fse added a comment to T6755: libgcrypt: KEM API.

The API that you quote at the end is indeed what is comonly understood as how a KEM functions and is exactly what fits to ML-KEM.

Oct 10 2023, 9:11 AM · PQC, libgcrypt
gniibe triaged T6755: libgcrypt: KEM API as Wishlist priority.
Oct 10 2023, 8:23 AM · PQC, libgcrypt

Oct 9 2023

gniibe added a comment to T6637: PQC for Libgcrypt.

Please send us patches (to this branch).

Oct 9 2023, 10:29 AM · PQC, libgcrypt
fse added a comment to T6637: PQC for Libgcrypt.

One question on the future cooperation: is it from now on possible to directly commit to these branches or will we continue to work with uploading patches to this task?

Oct 9 2023, 8:18 AM · PQC, libgcrypt

Oct 6 2023

gniibe added a comment to T6637: PQC for Libgcrypt.

Pushed the change into kem-kyber branch.
https://dev.gnupg.org/source/libgcrypt/history/kem-kyber/

Oct 6 2023, 6:42 AM · PQC, libgcrypt

Oct 5 2023

werner triaged T6747: sexp string including \0 as Normal priority.
Oct 5 2023, 11:41 AM · libgcrypt, Bug Report
gniibe changed the status of T6747: sexp string including \0 from Open to Testing.
Oct 5 2023, 8:38 AM · libgcrypt, Bug Report
gniibe claimed T6747: sexp string including \0.
Oct 5 2023, 8:30 AM · libgcrypt, Bug Report
gniibe created T6747: sexp string including \0.
Oct 5 2023, 8:30 AM · libgcrypt, Bug Report
gniibe claimed T6637: PQC for Libgcrypt.

I'll create a branch for this work. Then, I'll incorporate changes to master.

Oct 5 2023, 7:15 AM · PQC, libgcrypt