Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash with sandboxing enabled on linux #12791

Open
fabricedesre opened this issue Aug 10, 2016 · 7 comments
Open

Crash with sandboxing enabled on linux #12791

fabricedesre opened this issue Aug 10, 2016 · 7 comments
Labels

Comments

@fabricedesre
Copy link
Contributor

@fabricedesre fabricedesre commented Aug 10, 2016

At least on Linux, we don't display anything but a grey background:
./mach run -r -b
/home/fabrice/dev/servo/target/release/build
ERROR:servo: expected message from layout: RecvError (thread PaintThread PipelineId { namespace_id: PipelineNamespaceId(0), index: PipelineIndex(0) }, at ../src/libcore/result.rs:788)
ERROR:servo: expected message from layout: RecvError (thread PaintThread PipelineId { namespace_id: PipelineNamespaceId(0), index: PipelineIndex(1) }, at ../src/libcore/result.rs:788)
ERROR:constellation::constellation: about:failure failed

Loading pages with ./mach -r http://mozilla.org works fine.

@paulrouget
Copy link
Contributor

@paulrouget paulrouget commented Aug 10, 2016

Can you try to run browserhtml without sandboxing and multiprocess?
Either run servo + browserhtml manually, or comment this line:

if not is_windows():

@fabricedesre
Copy link
Contributor Author

@fabricedesre fabricedesre commented Aug 10, 2016

Good catch. -M works without -S, but not with it.

@jdm
Copy link
Member

@jdm jdm commented Aug 10, 2016

@metajack @pcwalton Another instance of the sandboxing change causing bustage.

@metajack
Copy link
Contributor

@metajack metajack commented Aug 10, 2016

Are all examples on Linux?

@fabricedesre
Copy link
Contributor Author

@fabricedesre fabricedesre commented Aug 10, 2016

I only tested on Linux, so can't tell for other platforms.

@jdm
Copy link
Member

@jdm jdm commented Aug 10, 2016

So far yes.

@paulrouget
Copy link
Contributor

@paulrouget paulrouget commented Aug 15, 2016

Let's disable sandboxing (#12867) and keep this issue open

bors-servo added a commit that referenced this issue Aug 15, 2016
Revert "Enable multiprocess and sandbox"

Multiprocess and sandboxing still have some issues:
* #12791
* #12807

Let's disable these features for now and reenable them later: #12868.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12867)
<!-- Reviewable:end -->
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this issue Oct 1, 2019
…paulrouget:noMS); r=emilio

Multiprocess and sandboxing still have some issues:
* servo/servo#12791
* servo/servo#12807

Let's disable these features for now and reenable them later: servo/servo#12868.

Source-Repo: https://github.com/servo/servo
Source-Revision: a617ea353a059486a117e1c04a60e3d13fc5f9cc

UltraBlame original commit: 166e02c3b7a1db2ef2989dc0c2e12fe05909364a
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this issue Oct 1, 2019
…paulrouget:noMS); r=emilio

Multiprocess and sandboxing still have some issues:
* servo/servo#12791
* servo/servo#12807

Let's disable these features for now and reenable them later: servo/servo#12868.

Source-Repo: https://github.com/servo/servo
Source-Revision: a617ea353a059486a117e1c04a60e3d13fc5f9cc

UltraBlame original commit: 166e02c3b7a1db2ef2989dc0c2e12fe05909364a
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this issue Oct 1, 2019
…paulrouget:noMS); r=emilio

Multiprocess and sandboxing still have some issues:
* servo/servo#12791
* servo/servo#12807

Let's disable these features for now and reenable them later: servo/servo#12868.

Source-Repo: https://github.com/servo/servo
Source-Revision: a617ea353a059486a117e1c04a60e3d13fc5f9cc

UltraBlame original commit: 166e02c3b7a1db2ef2989dc0c2e12fe05909364a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.