[FLINK-22376] SequentialChannelStateReaderImpl may recycle buffer twice

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 1.13.0
• Fix Version/s: 1.14.0, 1.13.2
• Component/s: Runtime / Network, Runtime / Task
• Labels: pull-request-available
• Assignee: Anton Kalashnikov
• Reporter: Roman Khachatryan
• Created: 20/Apr/21 14:22
• Updated: 23/Sep/21 17:23
• Resolved: 07/Jun/21 09:03

Description

In ChannelStateChunkReader.readChunk in case of error buffer is recycled in the catch block. However, it might already have been recycled in stateHandler.recover().

Using minor priority, as this only affects already failing path.
 

Issue Links

• causes: FLINK-23724 Network buffer leak when ResultPartition is released (failover)
• links to: GitHub Pull Request #15885
• links to: GitHub Pull Request #16079