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

fix: stack overflow in bug report #1704

Merged
merged 3 commits into from
Oct 19, 2023
Merged

fix: stack overflow in bug report #1704

merged 3 commits into from
Oct 19, 2023

Conversation

lamonato29
Copy link
Contributor

fix the FS.mkdir_p(path) function that was causing stackoverflow exception

Description

I made a change so that the code works as follows: The path is divided into parts, then the function goes through the components, creating the directory if it does not exist. The process repeats until all components of the path have been successfully completed, avoiding stackoverflow exception. At the end, the function returns true to indicate that the operation was successful.

Please delete options that are not relevant.

  • [x ] Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested

  • [x ] Reported successfully using CTRL + Z command. The txt file with the message was created.

fix the FS.mkdir_p(path) function that was causing stackoverflow exception
@luan luan changed the title Fix: Report Bug fix: stack overflow in bugreport Oct 18, 2023
@luan luan changed the title fix: stack overflow in bugreport fix: stack overflow in bug report Oct 18, 2023
@luan
Copy link
Contributor

luan commented Oct 18, 2023

@lamonato29 can you give github actions permissions to write to your fork so it can push up the formatter changes?

@lamonato29
Copy link
Contributor Author

Done!

@luan luan merged commit f219099 into opentibiabr:main Oct 19, 2023
@lamonato29 lamonato29 deleted the FIX-OnReportBug branch October 22, 2023 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants