commit 1ea083d8b29b464727b8e0b43e4a65d068074573
author Adam Langley <agl@google.com> Wed Dec 12 10:46:13 2018 -0800
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> Wed Dec 12 18:58:50 2018 +0000
tree 90d0300345f090ac33c1f80bc4edf7d6c74e9dd0
parent 2526c66b72892b32e0477aef124915d8590e9762

Always 16-byte align |poly| elements.

Even if the vector code isn't used in hrss.c, it might call external
assembly that still requires alignment.

Change-Id: I11ceb88f96deec6b20883872030ca090506ca150
Reviewed-on: https://boringssl-review.googlesource.com/c/33584
Reviewed-by: Adam Langley <agl@google.com>
Commit-Queue: Adam Langley <agl@google.com>

crypto/hrss/hrss.c

diff --git a/crypto/hrss/hrss.c b/crypto/hrss/hrss.c
index c059b83..b7002c1 100644
--- a/crypto/hrss/hrss.c
+++ b/crypto/hrss/hrss.c
@@ -1022,7 +1022,9 @@
     vec_t vectors[VECS_PER_POLY];
   };
 #else
-  uint16_t v[N + 3];
+  // Even if !HRSS_HAVE_VECTOR_UNIT, external assembly may be called that
+  // requires alignment.
+  alignas(16) uint16_t v[N + 3];
 #endif
 };