diff options
| author | Bill Richardson <wfrichar@chromium.org> | 2015-01-26 12:18:25 -0800 |
|---|---|---|
| committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2015-01-28 01:55:58 +0000 |
| commit | 73e5eb38821d693244f841ce4f0a14546e5b6361 (patch) | |
| tree | 88f27661515c24c23a02eddca856433189f4b6f8 /tests/vb2_secdata_tests.c | |
| parent | 9a8dfd00ecf042b7619f0fbbcb8308fce5cfd5c8 (diff) | |
| download | vboot-73e5eb38821d693244f841ce4f0a14546e5b6361.tar.gz | |
vboot2: fix alignment issues on 32-bit architectures
We were assuming 8-byte alignment for buffers. That's not true on
32-bit architectures. We should make the alignment requirements
explicit (and correct) for all architectures.
BUG=chromium:452179
BRANCH=ToT
CQ-DEPEND=CL:243380
TEST=manual
USE=vboot2 FEATURES=test emerge-x86-alex vboot_reference
Change-Id: I120f23e9c5312d7c21ff9ebb6eea2bac1e430e37
Signed-off-by: Bill Richardson <wfrichar@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/243362
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Diffstat (limited to 'tests/vb2_secdata_tests.c')
| -rw-r--r-- | tests/vb2_secdata_tests.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/vb2_secdata_tests.c b/tests/vb2_secdata_tests.c index 51283317..d4344b0b 100644 --- a/tests/vb2_secdata_tests.c +++ b/tests/vb2_secdata_tests.c @@ -29,7 +29,8 @@ static void test_changed(struct vb2_context *ctx, int changed, const char *why) static void secdata_test(void) { - uint8_t workbuf[VB2_WORKBUF_RECOMMENDED_SIZE]; + uint8_t workbuf[VB2_WORKBUF_RECOMMENDED_SIZE] + __attribute__ ((aligned (VB2_WORKBUF_ALIGN))); struct vb2_context c = { .flags = 0, .workbuf = workbuf, |