Closed Bug 587000 Opened 14 years ago Closed 14 years ago

Crash on startup with "Library ssl3 not found"

Categories

(Core Graveyard :: Widget: Android, defect)

defect
Not set
normal

Tracking

(blocking2.0 beta4+, fennec2.0a1+)

RESOLVED FIXED
Tracking Status
blocking2.0 --- beta4+
fennec 2.0a1+ ---

People

(Reporter: mbrubeck, Assigned: mbrubeck)

Details

Attachments

(1 file)

The 2010-08-13 Android mozilla-central nightly crashes on startup with:

E/AndroidRuntime( 9091): FATAL EXCEPTION: main
E/AndroidRuntime( 9091): java.lang.UnsatisfiedLinkError: Library ssl3 not found
E/AndroidRuntime( 9091): 	at java.lang.Runtime.loadLibrary(Runtime.java:461)
E/AndroidRuntime( 9091): 	at java.lang.System.loadLibrary(System.java:557)
E/AndroidRuntime( 9091): 	at org.mozilla.gecko.GeckoAppShell.loadGeckoLibs(GeckoAppShell.java:132)
tracking-fennec: --- → ?
tracking-fennec: ? → 2.0a1+
From hg bisect, this appears to be a regression from Bug 576760.
Blocks: 576760
My local hg bisect (In reply to comment #1)
> From hg bisect, this appears to be a regression from Bug 576760.

Nevermind, further bisecting does not bear this out.  (I used an incremental tinderbox build for one starting point and a nightly build for the other, but it looks like the incremental build might have hidden the breakage from some earlier changeset.)
No longer blocks: 576760
This is almost certainly caused by bug 580679. libssl3.so now imports zlib stuff like "deflate", but doesn't actually link against -lz (and I'm not sure we want to be linking against the system-zlib in the first place). I'm confused how that patch was actually meant to work, and it should probably be backed out.
blocking2.0: --- → beta4+
This should be easy to fix.  Matt, if you work in
Mozilla's office in Mountain View, I'll be happy
to come over this afternoon to fix this.  (I don't
know how to build on Android.)  It's also easy to
disable the TLS zlib compression code for Android
only.
OS: Android → All
(In reply to comment #4)
> Matt, if you work in Mozilla's office in Mountain View, I'll be happy
> to come over this afternoon to fix this.

I'm not in Mountain View, but Michael Wu will be there this afternoon and can work with you on the Android bits.  Or you can find us on irc.mozilla.org.
Matt, Michael:

This patch for security/manager/Makefile.in has three "hunks":
http://hg.mozilla.org/mozilla-central/diff/1117034e2762/security/manager/Makefile.in

Could you try reverting just the last hunk and testing it
on Android?

After you make that change, do a "make clean" in
objdir/security/manager first, before issuing the top-level
make.  Thanks.
(In reply to comment #6)
> This patch for security/manager/Makefile.in has three "hunks":
> http://hg.mozilla.org/mozilla-central/diff/1117034e2762/security/manager/Makefile.in
> 
> Could you try reverting just the last hunk and testing it
> on Android?

Reverting the last hunk of that patch fixes the startup issue on Android.
Attached patch patchSplinter Review
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
Attachment #465738 - Flags: review?
Attachment #465738 - Flags: review? → review?(benjamin)
Attachment #465738 - Flags: review?(benjamin) → review+
Comment on attachment 465738 [details] [diff] [review]
patch

Matt,

Can you attach this patch to bug 580679 instead?  You can reopen
that bug.

The comment

+# Turn off TLS compression support because NSS 3.12.6 can't be built
+# with Mozilla's zlib.h.  See bug 527659 comment 10.

needs to be updated because you're turning if off for a different
reason now.

I'd prefer that you ifdef this with ANDROID.  Thanks.
Moved the patch to bug 580679 and updated the comment; NOT ifdef'ed with ANDROID, per bsmedberg's request on IRC.
pushed: http://hg.mozilla.org/mozilla-central/rev/0b20b89a66cd
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: