Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Firefox's GeckoLinker has intermittent crashes #5
Comments
thestinger
added
the
Type: bug
label
Aug 17, 2015
thestinger
added this to the Beta release milestone
Aug 19, 2015
thestinger
added
needs research
and removed
needs research
labels
Aug 23, 2015
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
polyzen
commented
Sep 17, 2015
|
Just adding a note that this also affects Fennec F-droid |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
thestinger
Sep 17, 2015
Contributor
I'm still not sure why this doesn't work, but I suspect that their custom ELF loader/linker is incompatible with a hardening feature that I'm not going to want to remove for the sake of one very screwed up application. Need to narrow down the cause before I can decide what to do though.
|
I'm still not sure why this doesn't work, but I suspect that their custom ELF loader/linker is incompatible with a hardening feature that I'm not going to want to remove for the sake of one very screwed up application. Need to narrow down the cause before I can decide what to do though. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
thestinger
Sep 17, 2015
Contributor
Also not really a priority for me because it lacks sandboxing and other hardening features. The default browser will end up being CyanogenMod's new one, which is a fork of Chromium with some added features like built-in ad-blocking.
|
Also not really a priority for me because it lacks sandboxing and other hardening features. The default browser will end up being CyanogenMod's new one, which is a fork of Chromium with some added features like built-in ad-blocking. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
thestinger
Sep 17, 2015
Contributor
Would be nice if they provided a way to use the normal linker... I expect that it'd work fine.
|
Would be nice if they provided a way to use the normal linker... I expect that it'd work fine. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
vanitasvitae
commented
Sep 26, 2015
|
It seems like Orfox (based on Firefox) also doesnt work. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
thestinger
Sep 27, 2015
Contributor
Yeah, that's expected. I'll spend some time narrowing down the cause of this problem in the next week or two. I have a strong feeling that it's an incompatibility between a hardening feature and their linker though, and I'm not going to want to revert it for their sake. Perhaps they can be convinced to use the regular linker in some cases but I don't think they're going to care... :[
|
Yeah, that's expected. I'll spend some time narrowing down the cause of this problem in the next week or two. I have a strong feeling that it's an incompatibility between a hardening feature and their linker though, and I'm not going to want to revert it for their sake. Perhaps they can be convinced to use the regular linker in some cases but I don't think they're going to care... :[ |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
thestinger
Oct 15, 2015
Contributor
Starting to make some good progress on this. It appears there are two issues here. There's a crash due to a probably intentional memory access error that they intend to catch. I reverted a bunch of features to narrow it down, and I successfully avoided it. There's another intermittent start-up error that's not actually a crash, just an abort/exit, but I'll file a new bug for that when the crash is dealt with.
|
Starting to make some good progress on this. It appears there are two issues here. There's a crash due to a probably intentional memory access error that they intend to catch. I reverted a bunch of features to narrow it down, and I successfully avoided it. There's another intermittent start-up error that's not actually a crash, just an abort/exit, but I'll file a new bug for that when the crash is dealt with. |
thestinger
changed the title from
Firefox doesn't start
to
Firefox crashes at start-up with a SEGV_ACCERR signal
Oct 15, 2015
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
thestinger
Oct 16, 2015
Contributor
This is fixed and will be included in the next set of releases (likely tonight).
|
This is fixed and will be included in the next set of releases (likely tonight). |
thestinger
closed this
Oct 16, 2015
thestinger
changed the title from
Firefox crashes at start-up with a SEGV_ACCERR signal
to
Firefox's GeckoLinker has intermittent crashes
Oct 16, 2015
thestinger
added
upstream
upstream-app
and removed
upstream
labels
Oct 16, 2015
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
thestinger
Oct 17, 2015
Contributor
Orfox will still be broken since I only worked around the issue for the Firefox and Firefox Beta app names, so I'll have to deal with that too.
|
Orfox will still be broken since I only worked around the issue for the Firefox and Firefox Beta app names, so I'll have to deal with that too. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
vanitasvitae
commented
Oct 17, 2015
|
It would be nice if Fennec would be fixed too. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
thestinger
Oct 17, 2015
Contributor
Yeah, that will be fixed too, I just exported the environment variables globally since it's hard to know how many Firefox-based apps there are.
|
Yeah, that will be fixed too, I just exported the environment variables globally since it's hard to know how many Firefox-based apps there are. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
polyzen
commented
Oct 19, 2015
|
Firefox seems to work fine. Fennec is still crashing. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
thestinger
Oct 19, 2015
Contributor
I know, I haven't released a new ROM with the fix for Fennec and Orfox yet. It is fixed though.
|
I know, I haven't released a new ROM with the fix for Fennec and Orfox yet. It is fixed though. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment|
The new ROM is published for the Nexus 5 now. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
vanitasvitae
commented
Oct 19, 2015
|
Works for me for Firefox, Fennec, Orfox. Nice :) |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
polyzen
Oct 19, 2015
Thank you :)
On Mon, Oct 19, 2015 at 5:12 PM, vanitasvitae notifications@github.com
wrote:
Works for me for Firefox, Fennec, Orfox. Nice :)
—
Reply to this email directly or view it on GitHub
copperhead#5 (comment).
polyzen
commented
Oct 19, 2015
|
Thank you :) On Mon, Oct 19, 2015 at 5:12 PM, vanitasvitae notifications@github.com
|
thestinger commentedAug 17, 2015
It's not entirely clear what's wrong but it's probably caused by Firefox's custom ELF loader/linker (GeckoLinker). It does crazy things like intentionally causing segmentation faults for lazy library loading and it monkey patches functions in libc.