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

[TW#16750] esp_wifi_80211_tx function overwrites frame counter #1332

Closed
seppestas opened this issue Nov 29, 2017 · 2 comments
Closed

[TW#16750] esp_wifi_80211_tx function overwrites frame counter #1332

seppestas opened this issue Nov 29, 2017 · 2 comments

Comments

@seppestas
Copy link
Contributor

seppestas commented Nov 29, 2017

I originally reported this as a comment on issue #677.

The esp_wifi_80211_tx function seems to overwrite the frame counter of the frames sent using it, even if en_sys_seq is set to false.

When setting en_sys_seq to false the frame counter is always overwritten to 0. When setting en_sys_seq to true the frame counter follows the frame counter of frames sent by other components acting on WiFi (e.g AP beacons), but the frames sent by esp_wifi_80211_tx do not themselves cause the frame counter to increment. This happens both when WiFi was configured in AP and Station mode. Tested on ESP-IDF v2.1 and on the master branch (094e47f).

The old ieee80211_freedom_output function did not have this behaviour, allowing to send frames with arbitrary frame counter values.

@FayeY FayeY changed the title esp_wifi_80211_tx function overwrites frame counter [TW#16750] esp_wifi_80211_tx function overwrites frame counter Dec 1, 2017
@liuzfesp
Copy link
Contributor

@seppestas esp_wifi_80211_tx will be optimized in v3.1

@seppestas
Copy link
Contributor Author

@liuzfesp Any idea when v3.1 would become available? The milestones on GH do not seem to be very up to date...

We currently have 2 different firmwares, one based on ESP-IDF v2.0 to support the low level IEEE802.11 functionality we need and one where we have a BLE PoC for configuration. Merging both to a newer version of the ESP-IDF would be a lot easier (and future proof) than trying to backport the BLE functionality.

@igrr igrr closed this as completed in 846b848 May 18, 2018
0xFEEDC0DE64 pushed a commit to 0xFEEDC0DE64/esp-idf that referenced this issue May 5, 2021
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

No branches or pull requests

2 participants