MessageExtEncoder#encode(MessageExtBrokerInner) 计算消息长度是否超过最大允许的消息长度，是不是有问题？

[coolmoon101]

RocketMQ 版本5.3.1，计算msgLen时并没有包含crc32的长度，最后写入了crc32的数据。

public PutMessageResult encode(MessageExtBrokerInner msgInner) {
// ......
// 计算消息总长度，这个长度并没有包含crc32的长度
final int msgLen = calMsgLength(
msgInner.getVersion(), msgInner.getSysFlag(), bodyLength, topicLength, propertiesLength);

    // 这里计算是不是有问题，改为 msgLen + 4 > his.maxMessageSize 呢？
    if (msgLen > this.maxMessageSize) {
        CommitLog.log.warn("message size exceeded, msg total size: " + msgLen + ", msg body size: " + bodyLength
            + ", maxMessageSize: " + this.maxMessageSize);
        return new PutMessageResult(PutMessageStatus.MESSAGE_ILLEGAL, null);
    }
    //  省略1-17的消息写入代码

    // 18 CRC32
    this.byteBuf.writerIndex(this.byteBuf.writerIndex() + crc32ReservedLength);

    return null;
}

[coolmoon101]

已经找到问题：propertiesLength 这里已经加上了 crc32ReservedLength 的长度
final int propertiesLength = (propertiesData == null ? 0 : propertiesData.length)
+ (needAppendLastPropertySeparator ? 1 : 0) + crc32ReservedLength;