Skip to content
New issue

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

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

Already on GitHub? # to your account

VM crash on simple program using stdin #1876

Closed
DartBot opened this issue Feb 27, 2012 · 8 comments
Closed

VM crash on simple program using stdin #1876

DartBot opened this issue Feb 27, 2012 · 8 comments
Assignees
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. closed-obsolete Closed as the reported issue is no longer relevant P3 A lower priority bug or feature request

Comments

@DartBot
Copy link

DartBot commented Feb 27, 2012

This issue was originally filed by rice@google.com


The following test program crashes on startup on OS X 10.5.8 using a dart vm build @­r4617.

import("dart:io");

void main() {
  var stream = new StringInputStream(stdin);
  stream.lineHandler = () {
    var s = stream.readLine();
    if (s != null) {
      print(s);
    }
  };
}

$ dart Test.dart
/Volumes/WS/dart/dart/runtime/bin/eventhandler_macos.cc:109: error: Failed updating kqueue
Abort trap

(gdb) run Test.dart
Starting program: /Users/rice/bin/dart Test.dart
Reading symbols for shared libraries ++++. done
/Volumes/WS/dart/dart/runtime/bin/eventhandler_macos.cc:109: error: Failed updating kqueue

Program received signal SIGABRT, Aborted.
0x0011359b in dart::HeapPage::Contains (this=0x5000000, addr=84055912) at pages.h:26
26 return memory_->Contains(addr);
(gdb) where

­0 0x0011359b in dart::HeapPage::Contains (this=0x5000000, addr=84055912) at pages.h:26

­1 0x00112840 in dart::PageSpace::Contains (this=0x802e00, addr=84055912) at /Volumes/WS/dart/dart/runtime/vm/pages.cc:177

­2 0x000a521b in dart::Heap::Contains (this=0x400f90, addr=84055912) at /Volumes/WS/dart/dart/runtime/vm/heap.cc:107

­3 0x0001b28e in dart::Object::SetRaw (this=0xbfffead8, value=0x5029769) at object.h:3709

­4 0x0001b619 in dart::Library::initializeHandle (obj=0xbfffead8, raw_ptr=0x5029769) at object.h:1790

­5 0x0001b680 in dart::Library::Handle (islt=0x803800, raw_ptr=0x5029769) at object.h:1790

­6 0x0001b6a2 in dart::Library::Handle (raw_ptr=0x5029769) at object.h:1790

­7 0x000cd704 in dart::Function::ImplicitClosureFunction (this=0xbfffe9f0) at /Volumes/WS/dart/dart/runtime/vm/object.cc:3442

­8 0x00075b80 in dart::DRT_HelperResolveImplicitClosureFunction (isolate=0x803800, arguments={isolate_ = 0x803800, argc_ = 2, argv_ = 0xbfffeba0, retval_ = 0xbfffeba4}) at /Volumes/WS/dart/dart/runtime/vm/code_generator.cc:713

­9 0x00075d99 in dart::DRT_ResolveImplicitClosureFunction (arguments={isolate_ = 0x803800, argc_ = 2, argv_ = 0xbfffeba0, retval_ = 0xbfffeba4}) at /Volumes/WS/dart/dart/runtime/vm/code_generator.cc:684

­10 0x005400e2 in ?? ()

­11 0x005403f0 in ?? ()

­12 0x050c7f6f in ?? ()

­13 0x050c09e2 in ?? ()

­14 0x050c007d in ?? ()

­15 0x00090071 in dart::DartEntry::InvokeStatic (function=@0x843c48, arguments=@0xbffff308, optional_arguments_names=@0x843c50) at /Volumes/WS/dart/dart/runtime/vm/dart_entry.cc:88

­16 0x0000ac5a in Dart_InvokeStatic (library_in=0x805040, class_name_in=0x805048, function_name_in=0x805044, number_of_arguments=0, arguments=0x0) at /Volumes/WS/dart/dart/runtime/vm/dart_api_impl.cc:2027

­17 0x00003d9f in main (argc=2, argv=0xbffff45c) at /Volumes/WS/dart/dart/runtime/bin/main.cc:604

(gdb)

@DartBot
Copy link
Author

DartBot commented Feb 27, 2012

This comment was originally written by rice@google.com


Added Area-VM label.

@iposva-google
Copy link
Contributor

I was not able to reproduce this on Mac OS X 10.7.3. How reproducible is this in your setup?


Set owner to rice@google.com.

@iposva-google
Copy link
Contributor

Added Triaged label.

@DartBot
Copy link
Author

DartBot commented Feb 27, 2012

This comment was originally written by rice@google.com


I can repro every time.

@iposva-google
Copy link
Contributor

This only reproduces on 10.5.8, so it is likely something to do with kqueue usage. As we should not continue to support 10.5 much longer I am adjusting the priority. Still it would be good to know if we are using the kqueue apis in a wrong fashion.


Set owner to @madsager.
Removed Priority-Medium label.
Added Priority-Low label.

@iposva-google
Copy link
Contributor

Added Accepted label.

@madsager
Copy link
Contributor

I have made the error message printed for these types of errors more informative. I cannot reproduce this. If someone could run this with a tip of tree Dart VM and post the resulting error message here that might provide a hint. Thanks.

@madsager
Copy link
Contributor

madsager commented Oct 5, 2012

I have never been able to reproduce this. Closing since I cannot take action on this. Please reopen if you see this again.


Added AssumedStale label.

@DartBot DartBot added Type-Defect P3 A lower priority bug or feature request area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. closed-obsolete Closed as the reported issue is no longer relevant labels Oct 5, 2012
copybara-service bot pushed a commit that referenced this issue Jan 10, 2023
…_server, http_parser, logging, mockito, path, pool, shelf, source_map_stack_trace, sse, stream_channel, term_glyph, test, typed_data, watcher, web_socket_channel, webdev, webkit_inspection_protocol, yaml

Revisions updated by `dart tools/rev_sdk_deps.dart`.

cli_util (https://github.com/dart-lang/cli_util/compare/5a8e8ee..32bc077):
  32bc077  2023-01-09  Sam Rawlins  Migrate from no-implicit-casts to strict-casts (#73)

clock (https://github.com/dart-lang/clock/compare/6b8b7bf..65e8a13):
  65e8a13  2023-01-09  Sam Rawlins  Migrate from no-implicit-casts to strict-casts (#42)

csslib (https://github.com/dart-lang/csslib/compare/d776535..7054945):
  7054945  2023-01-09  Sam Rawlins  Migrate from no-implicit-casts to strict-casts (#157)

dartdoc (https://github.com/dart-lang/dartdoc/compare/29a1bbf..c4ab682):
  c4ab6824  2023-01-10  Sam Rawlins  Regenerate so latest analyzer is happy (#3293)

fixnum (https://github.com/dart-lang/fixnum/compare/714381c..71f0d4d):
  71f0d4d  2023-01-09  Sam Rawlins  Migrate from no-implicit-casts to strict-casts (#103)

glob (https://github.com/dart-lang/glob/compare/7adf833..4579281):
  4579281  2023-01-09  Sam Rawlins  Migrate from no-implicit-casts to strict-casts (#68)

http_multi_server (https://github.com/dart-lang/http_multi_server/compare/beb40a7..cce5080):
  cce5080  2023-01-09  Sam Rawlins  Migrate from no-implicit-casts to strict-casts (#48)

http_parser (https://github.com/dart-lang/http_parser/compare/16a4f34..6f73e4a):
  6f73e4a  2023-01-09  Sam Rawlins  Migrate from no-implicit-casts to strict-casts (#66)

logging (https://github.com/dart-lang/logging/compare/b525d5c..34ed68f):
  34ed68f  2023-01-09  Kevin Moore  Move to new analyzer language settings (#126)

mockito (https://github.com/dart-lang/mockito/compare/942dd03..9cc958a):
  9cc958a  2023-01-09  Sam Rawlins  Migrate from no-implicit-casts to strict-casts

path (https://github.com/dart-lang/path/compare/1299791..9768908):
  9768908  2023-01-09  Sam Rawlins  Migrate from no-implicit-casts to strict-casts (#133)

pool (https://github.com/dart-lang/pool/compare/713e631..ad4e2a7):
  ad4e2a7  2023-01-09  Sam Rawlins  Migrate from no-implicit-casts to strict-casts (#60)

shelf (https://github.com/dart-lang/shelf/compare/32e342d..a44e95e):
  a44e95e  2023-01-10  Kevin Moore  Fix doc comment references (#323)

source_map_stack_trace (https://github.com/dart-lang/source_map_stack_trace/compare/e5f9564..adea3e5):
  adea3e5  2023-01-09  Sam Rawlins  Migrate no-implicit-casts to strict-casts (#32)

sse (https://github.com/dart-lang/sse/compare/3c37edb..be426a2):
  be426a2  2023-01-09  Sam Rawlins  Migrate from no-implicit-casts to strict-casts (#74)

stream_channel (https://github.com/dart-lang/stream_channel/compare/0a7800a..3b99268):
  3b99268  2023-01-09  Sam Rawlins  Migrate from no-implicit-casts to strict-casts (#84)

term_glyph (https://github.com/dart-lang/term_glyph/compare/2bf4594..8cd9318):
  8cd9318  2023-01-09  Sam Rawlins  Migrate from no-implicit-casts to strict-casts (#31)

test (https://github.com/dart-lang/test/compare/3415089..932a652):
  932a652b  2023-01-09  Nate Bosch  Add a Map operator[] extension (#1830)
  1f2e963e  2023-01-09  Nate Bosch  Add Future.doesNotComplete condition (#1827)

typed_data (https://github.com/dart-lang/typed_data/compare/dbf81a7..9c209b9):
  9c209b9  2023-01-09  Sam Rawlins  Migrate no-implicit-casts to strict-casts (#58)

watcher (https://github.com/dart-lang/watcher/compare/2e0db71..3b49c7e):
  3b49c7e  2023-01-09  Sam Rawlins  Migrate from no-implicit-casts to strict-casts (#134)

web_socket_channel (https://github.com/dart-lang/web_socket_channel/compare/ebd0fe9..28d7b82):
  28d7b82  2023-01-10  Kevin Moore  Fix CI link in readme (#248)
  c6ce895  2023-01-09  Sam Rawlins  Update analysis_options.yaml (#247)

webdev (https://github.com/dart-lang/webdev/compare/49f97b8..094ee97):
  094ee97  2023-01-10  Anna Gringauze  Add back setExceptionPauseMode (#1871)
  2e65ddf  2023-01-10  Anna Gringauze  Prepare for dart 3.0 alpha breaking changes (#1880)
  2e2587a  2023-01-10  Elliott Brooks (she/her)  Add hotfix instructions to DWDS (#1876)
  fe39123  2023-01-10  Elliott Brooks (she/her)  Re-enable `reload_tests` (#1877)
  e71c0e3  2023-01-09  Anna Gringauze  Fix crash on processing devtools event (#1875)

webkit_inspection_protocol (https://github.com/google/webkit_inspection_protocol.dart/compare/ddb624c..15244ff):
  15244ff  2023-01-09  Kevin Moore  Add cron to CI (#97)

yaml (https://github.com/dart-lang/yaml/compare/02be51e..b2fce6c):
  b2fce6c  2023-01-10  Sam Rawlins  Migrate from no-implicit-casts to strict-casts (#134)

Change-Id: I4fb257fe7c697fab562f4d18e4ea50c5903b4cd8
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/278807
Auto-Submit: Devon Carew <devoncarew@google.com>
Reviewed-by: Samuel Rawlins <srawlins@google.com>
Commit-Queue: Samuel Rawlins <srawlins@google.com>
copybara-service bot pushed a commit that referenced this issue Jan 31, 2023
…g, mockito, package_config, shelf, string_scanner, test, webdev

Revisions updated by `dart tools/rev_sdk_deps.dart`.

async (https://github.com/dart-lang/async/compare/f700e9a..f700e9a):
  f700e9a  2023-01-27  Devon Carew  blast_repo fixes (#231)

characters (https://github.com/dart-lang/characters/compare/4526aa8..4526aa8):
  4526aa8  2023-01-30  Lasse R.H. Nielsen  Update tables to Unicode 15.0. (#71)

collection (https://github.com/dart-lang/collection/compare/a566328..a566328):
  a566328  2023-01-26  Devon Carew  add a publish script; prep to publish (#267)

dartdoc (https://github.com/dart-lang/dartdoc/compare/bc7bdc4..bc7bdc4):
  bc7bdc44  2023-01-30  dependabot[bot]  Bump js from 0.6.5 to 0.6.7 (#3310)

json_rpc_2 (https://github.com/dart-lang/json_rpc_2/compare/e73c4ad..e73c4ad):
  e73c4ad  2023-01-26  Devon Carew  blast_repo fixes (#89)

logging (https://github.com/dart-lang/logging/compare/399100a..399100a):
  399100a  2023-01-26  Devon Carew  add a publish script; prep to publish 1.1.1 (#128)

mockito (https://github.com/dart-lang/mockito/compare/d2a8df1..d2a8df1):
  d2a8df1  2023-01-30  Kevin Moore  Latest build_web_compilers, move to pkg:lints, fix breaks (#605)
  13340b5  2023-01-30  dependabot[bot]  Bump dart-lang/setup-dart from 1.3 to 1.4 (#600)

package_config (https://github.com/dart-lang/package_config/compare/3fe81c4..3fe81c4):
  3fe81c4  2023-01-30  Kevin Moore  Support latest pkg:build_web_compilers, lints. Update min SDK (#129)

shelf (https://github.com/dart-lang/shelf/compare/8fca9d9..8fca9d9):
  8fca9d9  2023-01-26  Devon Carew  blast_repo fixes (#326)

string_scanner (https://github.com/dart-lang/string_scanner/compare/29e471e..29e471e):
  29e471e  2023-01-30  dependabot[bot]  Bump dart-lang/setup-dart from 1.3 to 1.4 (#53)

test (https://github.com/dart-lang/test/compare/cec47c1..cec47c1):
  cec47c1c  2023-01-27  Nate Bosch  Add missing pub requirements (#1878)
  c99d455e  2023-01-27  Nate Bosch  Prepare to publish (#1877)
  0e7ec6a7  2023-01-27  Nate Bosch  Rename `Check` to `Subject` (#1875)
  78382731  2023-01-27  Nate Bosch  Add String.matches condition (#1874)
  26e0e87b  2023-01-27  Nate Bosch  Add Iterable.containsInOrder condition (#1873)
  c9232d6b  2023-01-27  Nate Bosch  Rename `that` to `which` (#1872)
  457166b3  2023-01-26  Nate Bosch  Add missing dependency on package:lints (#1876)
  193f2a0b  2023-01-26  Nate Bosch  Retry instead of extend timeout for flaky Node tests (#1871)
  7ad9b2c3  2023-01-26  Nate Bosch  Overhaul async matchers (#1868)
  ca254546  2023-01-26  Nate Bosch  Add a withQueue utility (#1870)
  6ae2e5e9  2023-01-26  Nate Bosch  Refactor tests to a new isRejectedBy utility (#1867)
  5aeba66d  2023-01-26  Nate Bosch  Use pubspec_overrides.yaml files (#1869)

webdev (https://github.com/dart-lang/webdev/compare/ce9c581..ce9c581):
  ce9c581  2023-01-29  Anna Gringauze  Validate only needed summaries in expression_compiler_service (#1920)

Change-Id: I3ddb0ddeb3b989f6f9e78cd8aa6327aba5899018
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/280078
Commit-Queue: Devon Carew <devoncarew@google.com>
Auto-Submit: Devon Carew <devoncarew@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
This issue was closed.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. closed-obsolete Closed as the reported issue is no longer relevant P3 A lower priority bug or feature request
Projects
None yet
Development

No branches or pull requests

3 participants