Skip to content
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

Closed
LorenzMeier opened this issue Nov 26, 2017 · 9 comments
Closed

Potential critical NuttX alignment issue #8367

LorenzMeier opened this issue Nov 26, 2017 · 9 comments

Comments

@LorenzMeier
Copy link
Member

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!

@LorenzMeier LorenzMeier added this to the Release v1.7.0 milestone Nov 26, 2017
@BazookaJoe1900
Copy link
Member

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.
I have seen the issue because I didn't set BOARD_HAS_CAPTURE, 'once' var has become the last one, and its uint8_t

@dagar
Copy link
Member

dagar commented Nov 26, 2017

@BazookaJoe1900
Copy link
Member

Yes, I think that adding
. = ALIGN(4);
Should to the work.
Can you confirm?

@davids5
Copy link
Member

davids5 commented Nov 26, 2017

I will check and confirm it in the AM tomorrow.

@LorenzMeier
Copy link
Member Author

@davids5 Did you have a chance to have a look?

@davids5
Copy link
Member

davids5 commented Nov 27, 2017

@LorenzMeier yes. Current build is not affected, but I will do a PR to avoid any future issues.

@LorenzMeier
Copy link
Member Author

Moving to v1.8.

@dagar
Copy link
Member

dagar commented Dec 6, 2017

Fixed in #8382.

@dagar dagar closed this as completed Dec 6, 2017
@BazookaJoe1900
Copy link
Member

@davids5 Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants