Skip to content

Limit the number of returned access technology settings#209

Open
kstribrnAmzn wants to merge 2 commits into
FreeRTOS:mainfrom
kstribrnAmzn:erdxList
Open

Limit the number of returned access technology settings#209
kstribrnAmzn wants to merge 2 commits into
FreeRTOS:mainfrom
kstribrnAmzn:erdxList

Conversation

@kstribrnAmzn

Copy link
Copy Markdown
Member

Description

Respect the maximum list size of eRDX configurations.

In the case of a cellular modem returning more eRDX settings through the AT+CEDRXS? query, the current library would write out of bounds. This would only happen in two cases:

  • misconfiguration - you should match the max eRDX settings to the chip through the CELLULAR_EDRX_LIST_MAX_SIZE macros
  • MITM attack - where the attacker intercepts the hardwired communication and returns a deliberately oversized response.

Test Steps

Unit test added

Checklist:

  • I have tested my changes. No regression in existing tests.
  • I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

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