bluetooth broken #525

Closed
zongo opened this Issue Dec 10, 2016 · 11 comments

Comments

Projects
None yet
3 participants
@zongo

zongo commented Dec 10, 2016

Steps To Reproduce The Bug
Download NMF26O.2016.12.07.07.47.51 factory image. Install as per recommended steps.
Install with twrp and latest OS same issue.
Pairing device: Jabra Wireless headsets
Bluetooth eventhoug enabled says disconnected in settings.
Any pairing with device will crash bluetooth share app.
Trying to pair after that display the error: "Could not pair with Jabra REVO because of an incorrect PIN or Passkey". No pop up screen to insert pin or passkey.
Reproducible everytime
CopperheadOS version
NMF26O.2016.12.07.07.47.51
Nougat 7.1.1
Device Model
Nexus 6P
adb logcats attached
logcat.txt

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Dec 11, 2016

Contributor

Thanks, the log is helpful. This is likely an out-of-bounds access in that bluetooth library detected by -fsanitize=bounds.

Contributor

thestinger commented Dec 11, 2016

Thanks, the log is helpful. This is likely an out-of-bounds access in that bluetooth library detected by -fsanitize=bounds.

@thestinger thestinger added this to the Release milestone Dec 11, 2016

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Dec 12, 2016

Contributor

Try the latest release (NMF26O.2016.12.12.04.03.44). It shouldn't crash anymore.

Contributor

thestinger commented Dec 12, 2016

Try the latest release (NMF26O.2016.12.12.04.03.44). It shouldn't crash anymore.

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Dec 12, 2016

Contributor

This bug will be kept open to track fixing the detected undefined access even though it should be successfully worked around now.

Contributor

thestinger commented Dec 12, 2016

This bug will be kept open to track fixing the detected undefined access even though it should be successfully worked around now.

@zongo

This comment has been minimized.

Show comment Hide comment
@zongo

zongo Dec 13, 2016

OTA update with latest, pairing is still failing.
Same issue with bluetooth share app is crashing.
Crashing happens with Jabra headset. Even though crash with bluetooth app, devices says headset is paired. When trying to play music through headset, it does not work.
Issue not happening with Marshmallow.
Logs attached.
log6.txt

zongo commented Dec 13, 2016

OTA update with latest, pairing is still failing.
Same issue with bluetooth share app is crashing.
Crashing happens with Jabra headset. Even though crash with bluetooth app, devices says headset is paired. When trying to play music through headset, it does not work.
Issue not happening with Marshmallow.
Logs attached.
log6.txt

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Dec 13, 2016

Contributor

Well, it's almost certainly the bounds sanitizer catching an out-of-bounds access, but it could be in any of the libraries statically linked into that one. I don't have any bluetooth devices to test with, so all I can do is take shots in the dark. I don't have room to keep around symbols for each build but I'll do it for the next set so I can translate those tracebacks into something useful.

Contributor

thestinger commented Dec 13, 2016

Well, it's almost certainly the bounds sanitizer catching an out-of-bounds access, but it could be in any of the libraries statically linked into that one. I don't have any bluetooth devices to test with, so all I can do is take shots in the dark. I don't have room to keep around symbols for each build but I'll do it for the next set so I can translate those tracebacks into something useful.

@zongo

This comment has been minimized.

Show comment Hide comment
@zongo

zongo Dec 13, 2016

Thank you for your help. This is not a show stopper anyways. Love the ROM. This is exactly what I was looking for. Donated 100 $ today. Thank you again for your time.

zongo commented Dec 13, 2016

Thank you for your help. This is not a show stopper anyways. Love the ROM. This is exactly what I was looking for. Donated 100 $ today. Thank you again for your time.

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Dec 13, 2016

Contributor

Thanks to reproducible builds this can probably actually be solved now:

[strcat@thinktank hw]$ addr2line -e bluetooth.default.so 000d5416
/proc/self/cwd/system/bt/stack/./sdp/sdp_api.c:361 (discriminator 1)
Contributor

thestinger commented Dec 13, 2016

Thanks to reproducible builds this can probably actually be solved now:

[strcat@thinktank hw]$ addr2line -e bluetooth.default.so 000d5416
/proc/self/cwd/system/bt/stack/./sdp/sdp_api.c:361 (discriminator 1)
@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Dec 13, 2016

Contributor

So it's really libbt-stack that needs the bounds exception, it just gets statically linked into the other library.

Contributor

thestinger commented Dec 13, 2016

So it's really libbt-stack that needs the bounds exception, it just gets statically linked into the other library.

@thestinger thestinger removed this from the Release milestone Dec 14, 2016

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Dec 17, 2016

Contributor

Please try the new NMF26Q.2016.12.17.07.29.30 release.

Contributor

thestinger commented Dec 17, 2016

Please try the new NMF26Q.2016.12.17.07.29.30 release.

@dark159123

This comment has been minimized.

Show comment Hide comment
@dark159123

dark159123 Dec 17, 2016

I can confirm bluetooth is now working, testing with Pebble+Gadgetbridge.
Thanks mate!

I can confirm bluetooth is now working, testing with Pebble+Gadgetbridge.
Thanks mate!

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Apr 8, 2017

Contributor
Contributor

thestinger commented Apr 8, 2017

@thestinger thestinger closed this Apr 8, 2017

@thestinger thestinger locked and limited conversation to collaborators Apr 8, 2017

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