Skip to content

Fix memory corruption (marker was getting overflowed because #29

New issue

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

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

Already on GitHub? # to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jmailloux
Copy link

Fixed memory corruption that was causing an abort when running this on MacOS. In FindVersion, marker was only 8 bytes. It needed to be 9 bytes because some strings were 8 characters long. That would make 9 bytes with the null terminator.
Also, fixed the no -rw option. Before, it would create 000, 001, ... folders, but all of the code folders would be in the output directory instead of the appropriate numbered directory.

some strings are 8 characters.  With the null chararacter, it
needed to be 9 chars.  It was only 8.  Also, fixed the non rw option.
@astand
Copy link
Owner

astand commented Jun 30, 2024

Hi @jmailloux thank you for participating

Please, check the latest v3.1 release. I was adding new features regarding generation output configuration flexibility and also stumbled with that problem. FindVersion is completely refactored, I believe that this issue fixed now.
https://github.com/astand/c-coderdbc/tree/v3.1

if the last message in the list was the biggest message
_CYC defines are generated even if cycle time is 0
f suffix is used in number constants to specify float
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants