pqcrypto icon indicating copy to clipboard operation
pqcrypto copied to clipboard

pqcrypto-rainbow test fails compilation

Open mouse07410 opened this issue 2 years ago • 2 comments

Relevant (IMHO) error

$ cargo test
.  .  .
error: use of deprecated constant `ffi::PQCLEAN_RAINBOWVCIRCUMZENITHAL_CLEAN_CRYPTO_PUBLICKEYBYTES`: Insecure cryptography, do not use in production
    --> pqcrypto-rainbow/src/ffi.rs:1183:40
     |
1183 |             let mut pk_alt = vec![0u8; PQCLEAN_RAINBOWVCIRCUMZENITHAL_CLEAN_CRYPTO_PUBLICKEYBYTES];
     |                                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
.  .  .
warning: `pqcrypto-rainbow` (lib test) generated 234 warnings (171 duplicates)
error: could not compile `pqcrypto-rainbow` due to 153 previous errors; 234 warnings emitted
warning: build failed, waiting for other jobs to finish...

mouse07410 avatar Jul 03 '22 03:07 mouse07410

You need to enable the insecure-cryptography feature. I might need to point this out in more places.

thomwiggers avatar Jul 03 '22 08:07 thomwiggers

I did not enable insecure crypto for the build. So, why is it being tested (and then fails)?

I'd suggest that if it's supposed to be disabled - it should be disabled completely: doesn't get compiled or tested, unless a special setting is given.

mouse07410 avatar Jul 03 '22 12:07 mouse07410

Closed by #43

thomwiggers avatar Sep 02 '22 07:09 thomwiggers