On 06/15/2015 04:23 PM, Stephan Mueller wrote: >> + /* In FIPS mode only allow key size minimum 2K */ >> > + if (fips_enabled && (mpi_get_size(key->n) < 256)) { > Considering my previous email, shouldn't that check rather be > > if (fips_enabled && > ((mpi_get_size(key->n) != 256) || (mpi_get_size(key->n) != 384)) I'm not familiar with the FIPS requirements. I checked the NIST recommendations witch states that RSA: |n| >= 2048 is acceptable. If FIPS allows 2K and 3K only then we need to change it. -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html