Pavel Tupitsyn created IGNITE-28110:
---------------------------------------

             Summary: Netty buffer leak in ClientInboundMessageHandler on 
invalid message
                 Key: IGNITE-28110
                 URL: https://issues.apache.org/jira/browse/IGNITE-28110
             Project: Ignite
          Issue Type: Bug
          Components: thin clients ai3
            Reporter: Pavel Tupitsyn
            Assignee: Pavel Tupitsyn
             Fix For: 3.2


In *channelRead* we don't close the buffer when throwing an exception due to 
invalid state.

{code}
 org.apache.ignite.lang.IgniteException: Unexpected message received before 
handshake completion
      at 
org.apache.ignite.client.handler.ClientInboundMessageHandler.channelRead(ClientInboundMessageHandler.java:424)
      at 
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:357)
      at 
io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:361)
      at 
io.netty.handler.codec.ByteToMessageDecoder.channelRead(ByteToMessageDecoder.java:325)
      at 
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:357)
      at 
io.netty.handler.flush.FlushConsolidationHandler.channelRead(FlushConsolidationHandler.java:152)
      at 
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:355)
      at 
io.netty.handler.timeout.IdleStateHandler.channelRead(IdleStateHandler.java:288)
      at 
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:355)
      at 
io.netty.channel.DefaultChannelPipeline$HeadContext.channelRead(DefaultChannelPipeline.java:1429)
      at 
io.netty.channel.DefaultChannelPipeline.fireChannelRead(DefaultChannelPipeline.java:918)
      at 
io.netty.channel.nio.AbstractNioByteChannel$NioByteUnsafe.read(AbstractNioByteChannel.java:176)
      at 
io.netty.channel.nio.AbstractNioChannel$AbstractNioUnsafe.handle(AbstractNioChannel.java:445)
      at 
io.netty.channel.nio.NioIoHandler$DefaultNioRegistration.handle(NioIoHandler.java:388)
      at 
io.netty.channel.nio.NioIoHandler.processSelectedKey(NioIoHandler.java:596)
      at 
io.netty.channel.nio.NioIoHandler.processSelectedKeysOptimized(NioIoHandler.java:571)
      at 
io.netty.channel.nio.NioIoHandler.processSelectedKeys(NioIoHandler.java:512)
      at io.netty.channel.nio.NioIoHandler.run(NioIoHandler.java:484)
      at 
io.netty.channel.SingleThreadIoEventLoop.runIo(SingleThreadIoEventLoop.java:225)
      at 
io.netty.channel.SingleThreadIoEventLoop.run(SingleThreadIoEventLoop.java:196)
      at 
io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:1195)
      at 
io.netty.util.internal.ThreadExecutorMap$2.run(ThreadExecutorMap.java:74)
      at 
io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
      at java.base/java.lang.Thread.run(Thread.java:833)
    
[2026-02-27T03:12:49,595][ERROR][org.apache.ignite.internal.runner.app.PlatformTestNodeRunner_4-network-worker-4][ResourceLeakDetector]
 LEAK: ByteBuf.release() was not called before it's garbage-collected. See 
https://netty.io/wiki/reference-counted-objects.html for more information.
    Recent access records: 
    #1:
      Hint: 'ClientInboundMessageHandler#0' will handle the message from this 
point.
      
io.netty.channel.DefaultChannelPipeline.touch(DefaultChannelPipeline.java:115)
      
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:344)
      
io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:361)
      
io.netty.handler.codec.ByteToMessageDecoder.channelRead(ByteToMessageDecoder.java:325)
      
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:357)
      
io.netty.handler.flush.FlushConsolidationHandler.channelRead(FlushConsolidationHandler.java:152)
      
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:355)
      
io.netty.handler.timeout.IdleStateHandler.channelRead(IdleStateHandler.java:288)
      
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:355)
      
io.netty.channel.DefaultChannelPipeline$HeadContext.channelRead(DefaultChannelPipeline.java:1429)
      
io.netty.channel.DefaultChannelPipeline.fireChannelRead(DefaultChannelPipeline.java:918)
      
io.netty.channel.nio.AbstractNioByteChannel$NioByteUnsafe.read(AbstractNioByteChannel.java:176)
      
io.netty.channel.nio.AbstractNioChannel$AbstractNioUnsafe.handle(AbstractNioChannel.java:445)
{code}

https://ci.ignite.apache.org/buildConfiguration/DevApacheIgnite3x_Platforms_Ignite27373dbapiUpgradeApacheIgnite3x_Test_PlatformTests_PythonTests_314t/10128056?buildTab=log&linesState=759.985&logView=flowAware&focusLine=988



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to