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

Memory alignment issue in ES #621

Closed
skliper opened this issue Apr 16, 2020 · 4 comments · Fixed by #645 or #678
Closed

Memory alignment issue in ES #621

skliper opened this issue Apr 16, 2020 · 4 comments · Fixed by #645 or #678
Assignees
Labels
Milestone

Comments

@skliper
Copy link
Contributor

skliper commented Apr 16, 2020

Is your feature request related to a problem? Please describe.

/home/jhageman/cFS/cFS-GitHub/cfe/fsw/cfe-core/src/es/cfe_es_shell.c: In function 'CFE_ES_ShellOutputCommand':
/home/jhageman/cFS/cFS-GitHub/cfe/fsw/cfe-core/src/es/cfe_es_shell.c:178: error: cast increases required alignment of target type
/home/jhageman/cFS/cFS-GitHub/cfe/fsw/cfe-core/src/es/cfe_es_shell.c:179: error: cast increases required alignment of target type
/home/jhageman/cFS/cFS-GitHub/cfe/fsw/cfe-core/src/es/cfe_es_shell.c:208: error: cast increases required alignment of target type
/home/jhageman/cFS/cFS-GitHub/cfe/fsw/cfe-core/src/es/cfe_es_shell.c:209: error: cast increases required alignment of target type

Describe the solution you'd like
Resolve bug.

Describe alternatives you've considered
None.

Additional context
Other shell issues, #484, #84, etc.

Requester Info
Jacob Hageman - NASA/GSFC

@skliper skliper added the bug label Apr 16, 2020
@skliper skliper added this to the 6.8.0 milestone Apr 16, 2020
@jphickey
Copy link
Contributor

I likely have something related to this in my stash of unmerged fixes, because I had something for #313 and #314 too.

@skliper
Copy link
Contributor Author

skliper commented Apr 16, 2020

Want to submit something for those three? Needed ASAP (blocking cert work).

@jphickey
Copy link
Contributor

Right now focusing on nasa/osal#285 stuff, but if it can wait until next week, then I can submit something.

@jphickey
Copy link
Contributor

Note that -Wcast-align was something we added recently, it never compiled cleanly on Vxworks/PPC. Whenever testing for vxworks I simply turned this warning off in the local build.

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