diff options
author | Jack Lloyd <[email protected]> | 2017-09-20 02:08:57 -0400 |
---|---|---|
committer | Jack Lloyd <[email protected]> | 2017-09-20 02:08:57 -0400 |
commit | 4dc1459c8f126a25167ddc6ebb4d88c175abc22d (patch) | |
tree | 22fd27db55f15497d8f4f50890ba5ca886f26434 | |
parent | bf519f75321eeab85ee6ace9014c03fed41c8812 (diff) |
Reorder calls to avoid UBSan false positive
Or that's what I assume it is since reordering makes it go away.
-rw-r--r-- | src/tests/unit_ecc.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/tests/unit_ecc.cpp b/src/tests/unit_ecc.cpp index 408b5040c..d79404a79 100644 --- a/src/tests/unit_ecc.cpp +++ b/src/tests/unit_ecc.cpp @@ -216,12 +216,13 @@ class NIST_Curve_Reduction_Tests : public Test std::vector<Test::Result> results; #if defined(BOTAN_HAS_NIST_PRIME_REDUCERS_W32) - results.push_back(random_redc_test("P-192", Botan::prime_p192(), Botan::redc_p192)); - results.push_back(random_redc_test("P-224", Botan::prime_p224(), Botan::redc_p224)); - results.push_back(random_redc_test("P-256", Botan::prime_p256(), Botan::redc_p256)); results.push_back(random_redc_test("P-384", Botan::prime_p384(), Botan::redc_p384)); + results.push_back(random_redc_test("P-256", Botan::prime_p256(), Botan::redc_p256)); + results.push_back(random_redc_test("P-224", Botan::prime_p224(), Botan::redc_p224)); + results.push_back(random_redc_test("P-192", Botan::prime_p192(), Botan::redc_p192)); #endif results.push_back(random_redc_test("P-521", Botan::prime_p521(), Botan::redc_p521)); + return results; } |