Skip to content

Fix CMD_SEND_CHANNEL_DATA payload validation - #3376

Open
VoxMesh wants to merge 1 commit into
meshcore-dev:devfrom
VoxMesh:fix/channel-data-payload-limit
Open

Fix CMD_SEND_CHANNEL_DATA payload validation#3376
VoxMesh wants to merge 1 commit into
meshcore-dev:devfrom
VoxMesh:fix/channel-data-payload-limit

Conversation

@VoxMesh

@VoxMesh VoxMesh commented Sep 7, 2026

Copy link
Copy Markdown

Fixes #3345

Validate CMD_SEND_CHANNEL_DATA payloads against MAX_GROUP_DATA_LENGTH instead of
MAX_CHANNEL_DATA_LENGTH.

This rejects 166-167 byte payloads with ERR_CODE_ILLEGAL_ARG before they reach
sendGroupData(), while leaving the receive-side MAX_CHANNEL_DATA_LENGTH check unchanged.

Verified:

  • 164-165 byte payloads pass the validation check
  • 166-168 byte payloads are rejected by the validation check
  • Heltec_v3_companion_radio_ble builds successfully

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant