You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@rocketmq.apache.org by GitBox <gi...@apache.org> on 2026/10/11 07:17:11 UTC

[GH] (rocketmq/fix/pop-max-msg-nums-bounds): Workflow run "CodeQL Analysis" failed!

The GitHub Actions job "CodeQL Analysis" on rocketmq.git/fix/pop-max-msg-nums-bounds has failed.
Run started by GitHub user zjncs (triggered by zjncs).

Head commit for run:
34934e955d855236fcb1952a26c04441e4eddfae / zjncs <18...@163.com>
fix(broker): reject pop requests with non-positive maxMsgNums

processRequest only validated maxMsgNums > 32. A request with 0 fell
through to the polling logic with a zero message budget instead of
being rejected, and a negative value made new GetMessageResult pass
the negative size to new ArrayList, throwing IllegalArgumentException
from deep inside the pop path instead of a clean INVALID_PARAMETER
response.

Validate maxMsgNums >= 1 next to the existing upper-bound check.

Report URL: https://github.com/apache/rocketmq/actions/runs/34573341465

With regards,
GitHub Actions via GitBox