Skip to content

Do not crash on devices without a LinkKey - #5

Open
asarium wants to merge 1 commit into
x2es:masterfrom
asarium:support_no_linkkey
Open

Do not crash on devices without a LinkKey#5
asarium wants to merge 1 commit into
x2es:masterfrom
asarium:support_no_linkkey

Conversation

@asarium

@asarium asarium commented May 24, 2022

Copy link
Copy Markdown

If a bluetooth device does not have a LinkKey, for example if it does
not support pairing, then the current code would crash with an unknown
section error.

This fixes that by handling that case gracefully and showing those
devices in a separate list to make it clear that these are not relevant
for this project.

If a bluetooth device does not have a LinkKey, for example if it does
not support pairing, then the current code would crash with an unknown
section error.

This fixes that by handling that case gracefully and showing those
devices in a separate list to make it clear that these are not relevant
for this project.
@dydyamotya

Copy link
Copy Markdown

Hello, @asarium. These really should be included in master branch. I've needed to solve the same problem manually. @x2es, please merge this PR.

@x2es

x2es commented Jan 27, 2023

Copy link
Copy Markdown
Owner

Thank you for contribution!
Will test it.

@x2es x2es added the TODO label Jan 27, 2023
@quackers19

Copy link
Copy Markdown

the works perfectly on my device, should be merged to main

@aperomsik

aperomsik commented Feb 23, 2023

Copy link
Copy Markdown

It does at least allow me to see the other devices... but for me, the ones with no pairing key are BLE devices, and it would be great if this tool could support them. There's info and another script linked from here... but that script claims to work in the other direction. Would be nice to be able to push the Linux keys to Windows for BLE devices.
info.txt

@Simon128

Simon128 commented Apr 4, 2023

Copy link
Copy Markdown

It seems like these are BT 5.1 devices. I've create an implementation at #14 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants