Ble spam #2003
Merged
Ble spam #2003
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Ninja-jr
changed the title
|
Can i test them ? |
Contributor
Author
Sure, feel free to pull it to a feature branch of yours and test it then let me know is something needs to be bettered or changed |
Fix apple payloads packet length to make them fully compatible with apple devices expect from genuine packets.
Fix sprintf(nullptr) nonnull warning in BLESerialService Newer GCC versions in PlatformIO treat nullptr as invalid for nonnull-marked parameters, causing random build failures on updated runners. Replace with vsnprintf(NULL, 0, ...) + 1 for safe size calculation — standard-compliant and functionally identical. No behavior change to BLE serial commands.
Reverting mac randomization. Weren't working or caused crashes and reboots. Needs nimble library update
Reverting mac randomization. Weren't working or caused crashes and reboots. Needs nimble library update
Ninja-jr
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed Changes
-. 15 new working Apple payloads (replacing patched ones):
· AirPods series (7 variants)
· Beats headphones (5 variants)
· Apple setup services (AppleTV, New Phone, Transfer Number, etc.)
· Apple Vision Pro
· "Spam All Apple" - cycles through all 15 payloads
· Individual payload selection for targeted spam
· All existing protocols (Windows, Samsung, Android, Custom) unchanged
· Same BLE cycling pattern that already works
· Backward compatibility (old menu items redirect to new system)
Types of Changes
· New Feature: Apple spam system with 15 payloads
· UI Improvement: Clean menu structure
· Code Cleanup: Removed duplicate functions, fixed NimBLE data format
Verification
· Compiles cleanly for all target boards
· Menu navigation works (BLE → Apple → options)
· Individual Apple payloads can be started/stopped
· "Spam All" includes Apple payloads
· Existing protocols still functional
Testing
Linked Issues
· Deleted patched Apple payloads (Applejuice/SourApple)
· Provides foundation for future Apple payload additions
User-Facing Change
· BLE menu: "Applejuice", "SourApple" deleted (patched, don't work)
· BLE menu: "Apple" added → submenu with 15 working payloads
Further Comments
· Uses same BLE timing spam for most modes (100ms advertise, 5ms gap)
· Architecture allows easy addition of more Apple payloads later
· If Apple popups don't work perfectly, timing can be adjusted
· Full code is modular - Apple spam in separate files