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.

Sign up for GitHub

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

Jump to bottom

Registering the mode_flags field breaks other fields #8

Closed
FelixEcker opened this issue Jun 26, 2023 · 1 comment
Closed

Registering the mode_flags field breaks other fields #8

FelixEcker opened this issue Jun 26, 2023 · 1 comment
Assignees
@FelixEcker
Labels
bug Something isn't working

Comments

@FelixEcker
Copy link
Owner

Registering the mode_flags in mb_execute.c#mb_exec_compile causes the value of the file field and potentially some other fields to break.
@FelixEcker FelixEcker added the bug Something isn't working label Jun 26, 2023
@FelixEcker FelixEcker self-assigned this Jun 26, 2023
@FelixEcker
Copy link
Owner Author

Fixed with commit e699e98

This bug is caused by reallocation done by the register functions. Any pointer that is stored for quick access to a member of a mb_file struct needs to be reassigned (for example using the Navigation Functions of mb_parse.h) because the reallocation breaks the old pointers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@FelixEcker FelixEcker

Labels
bug Something isn't working
Projects
Mariebuild
Status: Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@FelixEcker