-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
Potential critical NuttX alignment issue #8367
Comments
Note that for what I seen so far, for px4fmu there should be no issue, because the last var on the bss is io_timer_channel_stats, which is 4 byte aligned. |
Add ALIGN to the linker scripts? https://github.com/PX4/Firmware/blob/master/nuttx-configs/px4fmu-v2/scripts/ld.script#L143 |
Yes, I think that adding |
I will check and confirm it in the AM tomorrow. |
@davids5 Did you have a chance to have a look? |
@LorenzMeier yes. Current build is not affected, but I will do a PR to avoid any future issues. |
Moving to v1.8. |
Fixed in #8382. |
@davids5 Thanks! |
The full report is here and I'm including it to ensure that we're not forgetting about it:
https://bitbucket.org/nuttx/nuttx/issues/78/possible-bss-override-by-stack-coloration
@davids5 @dagar I would appreciate your review.
Thanks!
The text was updated successfully, but these errors were encountered: