wang-jiahua opened a new pull request, #10567:
URL: https://github.com/apache/rocketmq/pull/10567

   ### Which Issue(s) This PR Fixes
   
   Fixes #10052
   
   ### Brief Description
   
   addCk() already guards against buffer overflow with popCkMaxBufferSize, but 
addCkJustOffset() did not have this check. Under high-concurrency POP 
consumption with large messages, the buffer ConcurrentHashMap could grow 
without bound, eventually causing OOM in PopBufferMergeService.
   
   Fix: Add the same counter > popCkMaxBufferSize guard at the beginning of 
addCkJustOffset(), consistent with addCk().
   
   ### How Did You Test This Change
   
   Existing PopBufferMergeServiceTest (4 tests) all pass.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to