search

aviggiano / redis-roaring

Roaring Bitmaps for Redis
MIT License
348 stars 56 forks source link

Check for memory leaks on redis module #13

Closed aviggiano closed 7 years ago

aviggiano commented 7 years ago

Since running valgrind on redis does not report memory leaks on redis modules, we can't know for sure if this module is correct.

lemire commented 7 years ago

Do you have reasons to suspect a memory leak?

aviggiano commented 7 years ago

No, but I noticed that if I purposely forget to free some allocated memory on my redis module I don't see it as a memory leak on valgrind.

I don't think that it has anything to do with CRoaring or redis.