commit 29bb1a7ebe55102c90611c021a142fdb6e97f8d5
author David Benjamin <davidben@google.com> Fri Mar 08 18:54:57 2024 -0500
committer Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> Sat Mar 09 00:33:49 2024 +0000
tree 7ff355c03a0446ddfd9e628628c386963c16af69
parent a56407d27da6ebf63ae9817dc19587a0ae98ef4a

Make crypto_test build with -Wframe-larger-than=25344

One consumer builds libcrypto with -Wframe-larger-than=25344. They
don't seem to build crypto_test, but mimicking that limit will be easier
if we can just apply it across the board.

Change-Id: I619780809f7cf8ac915fe1a103965b5f2e6dcd50
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/66828
Auto-Submit: David Benjamin <davidben@google.com>
Reviewed-by: Bob Beck <bbe@google.com>
Commit-Queue: David Benjamin <davidben@google.com>

crypto/hrss/hrss_test.cc

diff --git a/crypto/hrss/hrss_test.cc b/crypto/hrss/hrss_test.cc
index 8c4d15f..a3814ac 100644
--- a/crypto/hrss/hrss_test.cc
+++ b/crypto/hrss/hrss_test.cc
@@ -492,8 +492,11 @@
   uint8_t kCanary[256];
   static_assert(sizeof(kCanary) % 32 == 0, "needed for alignment");
   memset(kCanary, 42, sizeof(kCanary));
-  alignas(32) uint8_t
-      scratch[sizeof(kCanary) + POLY_MUL_RQ_SCRATCH_SPACE + sizeof(kCanary)];
+
+  auto scratch_buf = std::make_unique<uint8_t[]>(
+      32 + sizeof(kCanary) + POLY_MUL_RQ_SCRATCH_SPACE + sizeof(kCanary));
+  uint8_t *scratch =
+      static_cast<uint8_t *>(align_pointer(scratch_buf.get(), 32));
   OPENSSL_memcpy(scratch, kCanary, sizeof(kCanary));
   OPENSSL_memcpy(scratch + sizeof(kCanary) + POLY_MUL_RQ_SCRATCH_SPACE, kCanary,
                  sizeof(kCanary));