Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong maximum number of LMKs in docs (IDFGH-11890) #12976

Closed
rguca opened this issue Jan 14, 2024 · 1 comment
Closed

Wrong maximum number of LMKs in docs (IDFGH-11890) #12976

rguca opened this issue Jan 14, 2024 · 1 comment
Assignees
Labels
Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally

Comments

@rguca
Copy link

rguca commented Jan 14, 2024

The docs state the maximum number of LMKs is 6:

* LMK of the paired device is used to encrypt the vendor-specific action frame with the CCMP method. The maximum number of different LMKs is six. If the LMK of the paired device is not set, the vendor-specific action frame will not be encrypted.

After some testing, this seems not to be true. The maximum number of LMKs is defined by CONFIG_ESP_WIFI_ESPNOW_MAX_ENCRYPT_NUM.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Jan 14, 2024
@github-actions github-actions bot changed the title Wrong maximum number of LMKs in docs Wrong maximum number of LMKs in docs (IDFGH-11890) Jan 14, 2024
@espressif-bot espressif-bot added Status: In Progress Work is in progress and removed Status: Opened Issue is new labels Mar 13, 2024
@zhangyanjiaoesp
Copy link
Collaborator

zhangyanjiaoesp commented Mar 13, 2024

@rguca yes, you are right, we will update the doc. Thanks for your correction.

@espressif-bot espressif-bot added Status: Reviewing Issue is being reviewed Status: Done Issue is done internally Resolution: NA Issue resolution is unavailable and removed Status: In Progress Work is in progress Status: Reviewing Issue is being reviewed labels Mar 13, 2024
DarkZeros pushed a commit to DarkZeros/esp-idf that referenced this issue May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally
Projects
None yet
Development

No branches or pull requests

3 participants