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 memory corruption when maximum number of parameters is exceeded #456

Merged
merged 1 commit into from
Jun 7, 2019

Conversation

jacobperron
Copy link
Member

If the maximum number is exceeded fail with an informative error message.

Fixes #419.

If the maximum number is exceeded fail with an informative error message.

Fixes #419.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
@jacobperron
Copy link
Member Author

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@jacobperron jacobperron merged commit ec8539b into master Jun 7, 2019
@delete-merged-branch delete-merged-branch bot deleted the jacob/fix_419 branch June 7, 2019 21:56
clalancette pushed a commit that referenced this pull request Jun 12, 2019
…456)

If the maximum number is exceeded fail with an informative error message.

Fixes #419.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
clalancette pushed a commit that referenced this pull request Jun 12, 2019
…456)

If the maximum number is exceeded fail with an informative error message.

Fixes #419.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Consistent and critical failure parsing large (>500 lines) YAML configuration files.
2 participants