FS#55557 - [android-tools] SIGSEGV when running adb keygen

Attached to Project: Community Packages
Opened by Henry Tung (compgamer89) - Friday, 08 September 2017, 12:55 GMT
Last edited by Anatol Pomozov (anatolik) - Saturday, 09 September 2017, 07:58 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Anatol Pomozov (anatolik)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Description:

Running adb keygen causes segfault. System otherwise up-to-date (can provide other package versions upon request).

This bug is blocking, since it prevents pairing with new devices (no host key can be generated, and server startup fails to generate a new key with the same segfault).

The resulting adbkey file exists, but has zero length. No public key file is created.

Additional info:
* Version: 8.0.0_r4-1 (tested on x86_64)
* Logfiles from command: strace -o adb-keygen-strace.log adb keygen adbkey >adb-keygen.log 2>&1


Steps to reproduce:
* mkdir ~/.android && cd ~/.android
* adb keygen adbkey
   adb-keygen.log (0.2 KiB)
   adb-keygen-strace.log (25 KiB)
This task depends upon

Closed by  Anatol Pomozov (anatolik)
Saturday, 09 September 2017, 07:58 GMT
Reason for closing:  Fixed
Comment by Henry Tung (compgamer89) - Friday, 08 September 2017, 12:59 GMT
Probably related: https://bugzilla.redhat.com/show_bug.cgi?id=1441234
Comment by Henry Tung (compgamer89) - Friday, 08 September 2017, 13:31 GMT
Suggestions: Either use the patch from the linked bug, or use https://android.googlesource.com/platform/external/boringssl/+/android-8.0.0_r4 instead of the base boringssl repo.
Comment by Anatol Pomozov (anatolik) - Friday, 08 September 2017, 22:06 GMT
It is indeed a mess with boringssl vs system's OpenSSL. I statically linked android-tools against BoringSSL and the crash has gone. android-tools-8.0.0_r4-2 just pushed to the repo. Please verify that it works as expected.
Comment by Bruno Santos (bms) - Saturday, 09 September 2017, 07:48 GMT
New package works as expected. Thanks

Loading...