Skip to content

Commit

Permalink
do not clear the arena relocator callback
Browse files Browse the repository at this point in the history
  • Loading branch information
biojppm committed Jun 1, 2024
1 parent 508923b commit bdd4498
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion changelog/current.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ Most of the changes are from the giant Parser refactor described below. Before g
- a node has both an anchor and an alias (eg [SR86](https://matrix.yaml.info/details/SR86.html) [SU74](https://matrix.yaml.info/details/SU74.html)).
- tags contain [invalid characters](https://yaml.org/spec/1.2.2/#tag-shorthands) `,{}[]` (eg [LHL4](https://matrix.yaml.info/details/LHL4.html),[U99R](https://matrix.yaml.info/details/U99R.html),[WZ62](https://matrix.yaml.info/details/WZ62.html)).
- [#PR433](https://github.com/biojppm/rapidyaml/pull/433) - Add fuzz testing using libfuzzer. This caught some corner cases mostly related to scalar filtering and arena relocation, which were not present prior to the parser refactor. After these fixes, there were over ~500M runs without a single crash; and it is reasonable to say that the library is considerably hardened to memory problems.


### New features

Expand Down
2 changes: 0 additions & 2 deletions src/c4/yml/event_handler_stack.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -67,8 +67,6 @@ struct EventHandlerStack

void _stack_finish_parse()
{
m_relocate_arena = nullptr;
m_relocate_arena_data = nullptr;
}

protected:
Expand Down

0 comments on commit bdd4498

Please sign in to comment.