search

darkhz / bluetuith

A TUI bluetooth manager for Linux.
MIT License
850 stars 21 forks source link

bluez: fix panic caused by unsafe type assertion #40

Closed ddelazerda closed 8 months ago

darkhz commented 8 months ago

@ddelazerda did you experience a panic before, with the current release?

ddelazerda commented 8 months ago

Yes, as soon as I launched bluetuith it would panic. With my changes it doesn't panic and works just fine.

darkhz commented 8 months ago

This pull-request will not be merged, because types must be asserted properly across the all property accessors. See #42, and if possible, test it and post your review in the new pull request.

ddelazerda commented 8 months ago

I tested your test branch and it does solve the issue. I don't know how dbus works so when I set to fix the issue I was just trying to make it not panic :)