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

Jump to bottom

VM SegFault when sending Type object that doesn't exist in receiver isolate. #15983

Closed
lrhn opened this issue Jan 9, 2014 · 6 comments
Closed

VM SegFault when sending Type object that doesn't exist in receiver isolate. #15983

lrhn opened this issue Jan 9, 2014 · 6 comments
Labels
area-vm Use area-vm for VM related issues, including code coverage, FFI, and the AOT and JIT backends. closed-duplicate Closed in favor of an existing report

Comments

@lrhn
Copy link
Member

lrhn commented Jan 9, 2014

Example code:
minicrash.dart:

    import "dart:isolate";
    main() {
     Isolate.spawnUri("minicrash-lib.dart", null, new ReceivePort().sendPort);
    }

minicrash-lib.dart:

    import "dart:isolate";
    class C {}
    main(_, p) { p.send(C); }

The result is a segmentation fault on the release VM.
On the debug VM, it prints:

runtime/vm/snapshot.cc:206: error: expected: !library_.IsNull()
Aborted (core dumped)
@nex3
Copy link
Member

nex3 commented Jan 15, 2014

We on the pub team (and our users) would really like this fixed; it makes debugging issues when isolates are involved much harder than it needs to be.

I thought there was already a bug for this, but I can't find it now.

@sethladd
Copy link
Contributor

This affects an open-source contributor trying to add functionality to Stagehand: https://github.com/google/stagehand/issues/192#issuecomment-74385551

@a-siva
Copy link
Contributor

a-siva commented Feb 17, 2015

This is a duplicate of https://code.google.com/p/dart/issues/detail?id=21398
After that fix this test now correctly throws an Illegal Argument exception

Invalid argument(s): Illegal argument in isolate message : (object is a regular Dart Instance)
#­0 _SendPortImpl._sendInternal (dart:isolate-patch/isolate_patch.dart:189)
#­1 _SendPortImpl.send (dart:isolate-patch/isolate_patch.dart:173)
#­2 main (file:///usr/local/google/home/asiva/workspace/dart-ws2/dart/tests/isolate/issue_15983_child.dart:5:9)
#­3 _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:253)
#­4 _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:142)

The bug that nweiz is referring to is https://code.google.com/p/dart/issues/detail?id=12435. I will make it a duplicate of this bug.

@a-siva
Copy link
Contributor

a-siva commented Feb 17, 2015

Issue #12435 has been merged into this issue.

@a-siva
Copy link
Contributor

a-siva commented Feb 17, 2015

Added Fixed label.

@iposva-google
Copy link
Contributor

Added Duplicate label.
Marked as being merged into #21398.

@lrhn lrhn added Type-Defect area-vm Use area-vm for VM related issues, including code coverage, FFI, and the AOT and JIT backends. closed-duplicate Closed in favor of an existing report labels Feb 17, 2015
@iposva-google iposva-google mentioned this issue Jun 3, 2015
Closed
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-vm Use area-vm for VM related issues, including code coverage, FFI, and the AOT and JIT backends. closed-duplicate Closed in favor of an existing report
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@nex3 @sethladd @kevmoo @lrhn @iposva-google @a-siva