Skip to content

fix sniffer deauth, fix channel range, some code cleanup #1470

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

Merged
merged 1 commit into from
Jul 11, 2025

Conversation

polprog
Copy link
Contributor

@polprog polprog commented Jul 11, 2025

Proposed Changes

This PR fixes the deauth sending in sniffer.

Types of Changes

bug fix

Verification

Run airmon or other wifi sniffer to see the deauth frames are being sent

Further Comments

Lot more work to be done to improve the sniffer performance. sniffer gets completely frozen when there is any larger traffic, like >100 pkts/sec. To fix this, this code should be divided into FreeRTOS tasks - one task for sniffer callback, one for saving packets to SD, one for updating the display.

@pr3y pr3y merged commit 858abfe into pr3y:main Jul 11, 2025
5 checks passed
@Huzzla101
Copy link

Huzzla101 commented Jul 11, 2025 via email

@Gosheto1234
Copy link
Contributor

i tested the sniffer and the ui seems kinda funky and fast redrawing which doesnt allow me to smoothly switch channels like before , can you look into this ? , i tested on m5stickcplus2

@polprog
Copy link
Contributor Author

polprog commented Jul 15, 2025

This is a tougher bug to crack. the temporary fix is to not redraw the entire display every second. There is a problem with accessing the SD card and the display, I noticed that doing display operations just after saving/reading from SD card causes the display operation to not work at all. The entire wifi sniffer should be rewritten and separated into at least two RTOS tasks. One that does saving to SD card and one that handles the display.

For now the fix is to comment out some of the unnecesary display stats or update the display less often

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.

4 participants