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

AV inside mono_local_cprop+0x23f #13056

Closed
bwadswor opened this issue Feb 19, 2019 · 1 comment

Comments

@bwadswor
Copy link
Contributor

@bwadswor bwadswor commented Feb 19, 2019

Steps to Reproduce

  1. Assemble this app. ("ilasm mono_local_cprop_av.il")
.assembly mono_local_cprop_av {}
.assembly extern mscorlib{auto}
.class mono_local_cprop_av {
.method static int32 Main() {
 .entrypoint
 .maxstack	12

    ldc.i8 4
    ldc.i8 2
   cgt
   ldc.i4 2
  div
 ret
}
}

  1. run said app.. "mono-sgen.exe mono_local_cprop_av.exe"
  2. observe AV

Current Behavior

Hits an access violation.

Expected Behavior

exit code 0.
Expected behavior occurs on full .net framework and coreclr.

On which platforms did you notice this

[ ] macOS
[ ] Linux
[X86] Windows

Version Used:

Freshly locally built x86 Windows Debug build. Also repros on 5.18.0.240 Release, but stack isn't so nice...

Mono JIT compiler version 6.1.0 (Visual Studio built mono)
Copyright (C) 2002-2014 Novell, Inc, Xamarin Inc and Contributors. www.mono-project.com
TLS: __thread
SIGSEGV: normal
Notification: Thread + polling
Architecture: x86
Disabled: none
Misc: softdebug
Interpreter: yes
Suspend: preemptive
GC: sgen (concurrent by default)

Stacktrace

 # ChildEBP RetAddr  
00 003af404 5036a866 mono_2_0_sgen!mono_local_cprop+0x23f [...\mono\mini\local-propagation.c @ 549]
01 003af600 50368edc mono_2_0_sgen!mini_method_compile+0x1306 [...\mono\mini\mini.c @ 3516]
02 003af664 5037b1cb mono_2_0_sgen!mono_jit_compile_method_inner+0x3c [...\mono\mini\mini.c @ 4021]
03 003af6f4 50378067 mono_2_0_sgen!mono_jit_compile_method_with_opt+0x6cb [...\mono\mini\mini-runtime.c @ 2479]
04 003af710 5037c051 mono_2_0_sgen!mono_jit_compile_method_jit_only+0x27 [...\mono\mini\mini-runtime.c @ 2542]
05 003af768 502b41ee mono_2_0_sgen!mono_jit_runtime_invoke+0x1c1 [...\mono\mini\mini-runtime.c @ 3082]
06 003af78c 502af619 mono_2_0_sgen!do_runtime_invoke+0x7e [...\mono\metadata\object.c @ 2993]
07 003af7a8 502b53af mono_2_0_sgen!mono_runtime_invoke_checked+0x49 [...\mono\metadata\object.c @ 3161]
...
@marek-safar

This comment has been minimized.

Copy link
Member

@marek-safar marek-safar commented Feb 19, 2019

/cc @vargaz

monojenkins added a commit that referenced this issue Feb 22, 2019
…ed (#13127)

[jit/cprop] insert dummy insn if first insn of basic block got replaced

Fixes #13056 & #13057
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.