camellia: [Camellia-redis-proxy] memory leak

Module: [camellia-redis-proxy] Version: [v1.1.11] Content: [memory leak]

After upgrading camellia to version 1.1.11, My service got some new errors: i.n.u.ResourceLeakDetector:319 - LEAK: ByteBuf.release() was not called before it's garbage-collected. See https://netty.io/wiki/reference-counted-objects.html for more information.
https://netty.io/wiki/reference-counted-objects.html
I never got it before

About this issue

Original URL
State: closed
Created a year ago
Comments: 18 (8 by maintainers)

Most upvoted comments

root cause： from 1.1.8, i optimize the BulkReply, use ByteBuf instead of byte[], so ReplyEncoder will write ByteBuf to client channel direct. but if the client connection disconnect before BulkReply write in, the ByteBuf in BulkReply will leak, no one will call release.

caojiajun on Jan 12, 2023

yes, let me close this issue

tasszz2k on Jan 31, 2023

I think the optimize ideas of BulkReply through direct buf is right. but I need find the right way to do it.

caojiajun on Jan 12, 2023

this optimize maybe rollback. i think it is very dangerous, easily trigger bugs.

caojiajun on Jan 12, 2023

i will fix this bug in 1.1.12, maybe i will release 1.1.12 today.

caojiajun on Jan 12, 2023