c9s / r3

libr3 is a high-performance path dispatching library. It compiles your route paths into a prefix tree (trie). By using the constructed prefix trie in the start-up time, you may dispatch your routes with efficiency
http://c9s.github.com/r3/bench.html
MIT License
813 stars 83 forks source link

Correcting leaks in tests #140

Closed bjosv closed 2 years ago

bjosv commented 2 years ago

A couple of testcases did not free its match_entry allocations, which triggers leak indications.

Visible when building and running tests using the leak sanitizer, i.e

CFLAGS="-fno-omit-frame-pointer -fsanitize=leak" cmake ..
make all test
c9s commented 2 years ago

@bjosv This one is conflicted, could you please take a look? thank you

bjosv commented 2 years ago

@c9s This PR is ready now, I fixed the conflicts.

bjosv commented 2 years ago

Added corrections for the newly merged IPv6 tests aswell, so now all tests passes the leak check.

c9s commented 2 years ago

omg you are so productive!

bjosv commented 2 years ago

When there is time and its fun.. :) Thank you for this library!