On 23/06/2021 11:49, Daniel P. Berrangé wrote:
> On Wed, Jun 23, 2021 at 11:41:56AM +0300, Dov Murik wrote:
...
>>
>> Related: I wondered if I could replace HASH_SIZE in:
>>
>>
>> /* hard code sha256 digest size */
>> #define HASH_SIZE 32
>>
>> typedef struct QEMU_PACKED SevHashTableEntry {
>> QemuUUID guid;
>> uint16_t len;
>> uint8_t hash[HASH_SIZE];
>> } SevHashTableEntry;
>>
>>
>> with some SHA256-related constant from crypto/hash.h, but I only found
>> the qcrypto_hash_digest_len(QCRYPTO_HASH_ALG_SHA256) function which
>> doesn't work for setting sizes of arrays at compile-time.
>>
>> Daniel: do you know what would be the proper way?
>
> We don't have any public constants right now - they're just hardcoded
> in hash.c struct. We could define public constants, and use those in
> the struct instead, as well as in other callers.
>
Thanks Daniel.
I see the exact same pattern in block/quorom.c (see HASH_LENGTH there).
I'll leave this change for a separate series.
-Dov
