8000 Fix read 32 bits accesses to modules to be unaligned by pguyot · Pull Request #1634 · atomvm/AtomVM · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
/ AtomVM Public

Fix read 32 bits accesses to modules to be unaligned #1634

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

pguyot
Copy link
Collaborator
@pguyot pguyot commented Apr 18, 2025

Modules can be unaligned when they are loaded from a binary and reads should therefore be unaligned. This yielded a crash on arm32v7 CI.

Also update arm32v7 CI to use bookworm debian image

These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).

SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

Modules can be unaligned when they are loaded from a binary and reads should
therefore be unaligned. This yielded a crash on arm32v7 CI.

Also update arm32v7 CI to use bookworm debian image

Signed-off-by: Paul Guyot <pguyot@kallisys.net>
@bettio bettio merged commit 95ac53c into atomvm:release-0.6 Apr 19, 2025
145 of 148 checks passed
@pguyot pguyot deleted the w16/fix-unaligned-accesses-for-modules branch April 19, 2025 12:41
bettio added a commit that referenced this pull request Apr 20, 2025
Merge fix from release-0.6 for unaligned memory access when loading modules
from binaries (see also #1634).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0