search

greatscottgadgets / ubertooth

Software, firmware, and hardware designs for Ubertooth
https://greatscottgadgets.com/ubertoothone/
GNU General Public License v2.0
1.92k stars 430 forks source link

le: Fixed CC2400 FIFO usage to allow BLE packets that are larger then 32 bytes #341

Closed dor-zusman closed 3 years ago

dor-zusman commented 5 years ago

Added some code to correctly fill-out the CC2400 FIFO while transmitting BLE and removed the size limitation that was placed as a workaround. This modified code was used in the BLEEDINGBIT presentation at BHEU18.

dor-zusman commented 5 years ago

@mikeryan any comment?

mikeryan commented 5 years ago

This looks pretty good but I haven't had a chance to test it before merging it.

jorants commented 4 years ago

I just incorporated this with the latest version of the firmware, see https://github.com/jorants/ubertooth. I think there was a small mistake with the size of adv_ind in bt_slave_le (which was still 32), and I changed the code to respect the set PA level. O, and I changes the PDU to ADV_NONCONN_IND, which seems to fit better, but maybe this should be a setting?

It seems to work well, i was able to send an exposure notification from the GAEN framework using https://github.com/jorants/ubertooth-exposure-notification , which would not be possible without the longer packages.